annotate auto/module @ 8548:9ffef6054abf quic

HTTP/3: fixed handling request body eof. While for HTTP/1 unexpected eof always means an error, for HTTP/3 an eof right after a DATA frame end means the end of the request body. For this reason, since adding HTTP/3 support, eof no longer produced an error right after recv() but was passed to filters which would make a decision. This decision was made in ngx_http_parse_chunked() and ngx_http_v3_parse_request_body() based on the b->last_buf flag. Now that since 0f7f1a509113 (1.19.2) rb->chunked->length is a lower threshold for the expected number of bytes, it can be set to zero to indicate that more bytes may or may not follow. Now it's possible to move the check for eof from parser functions to ngx_http_request_body_chunked_filter() and clean up the parsing code. Also, in the default branch, in case of eof, the following three things happened, which were replaced with returning NGX_ERROR while implementing HTTP/3: - "client prematurely closed connection" message was logged - c->error flag was set - NGX_HTTP_BAD_REQUEST was returned The change brings back this behavior for HTTP/1 as well as HTTP/3.
author Roman Arutyunyan <arut@nginx.com>
date Wed, 16 Sep 2020 18:59:25 +0100
parents 2c7a2d75938a
children 4b1299b1856a
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
6382
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
1
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
2 # Copyright (C) Ruslan Ermilov
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
3 # Copyright (C) Nginx, Inc.
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
4
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
5
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
6 case $ngx_module_type in
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
7 HTTP_*) ngx_var=HTTP ;;
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
8 *) ngx_var=$ngx_module_type ;;
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
9 esac
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
10
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
11
6383
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
12 if [ "$ngx_module_link" = DYNAMIC ]; then
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
13
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
14 for ngx_module in $ngx_module_name; do
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
15 # extract the first name
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
16 break
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
17 done
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
18
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
19 DYNAMIC_MODULES="$DYNAMIC_MODULES $ngx_module"
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
20 eval ${ngx_module}_SRCS=\"$ngx_module_srcs\"
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
21
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
22 eval ${ngx_module}_MODULES=\"$ngx_module_name\"
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
23
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
24 if [ -z "$ngx_module_order" -a \
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
25 \( "$ngx_module_type" = "HTTP_FILTER" \
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
26 -o "$ngx_module_type" = "HTTP_AUX_FILTER" \) ]
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
27 then
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
28 eval ${ngx_module}_ORDER=\"$ngx_module_name \
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
29 ngx_http_copy_filter_module\"
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
30 else
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
31 eval ${ngx_module}_ORDER=\"$ngx_module_order\"
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
32 fi
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
33
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
34 if test -n "$ngx_module_incs"; then
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
35 CORE_INCS="$CORE_INCS $ngx_module_incs"
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
36 fi
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
37
6804
2c7a2d75938a Configure: honor dependencies of dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6603
diff changeset
38 if test -n "$ngx_module_deps"; then
2c7a2d75938a Configure: honor dependencies of dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6603
diff changeset
39 NGX_ADDON_DEPS="$NGX_ADDON_DEPS $ngx_module_deps"
2c7a2d75938a Configure: honor dependencies of dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6603
diff changeset
40 fi
2c7a2d75938a Configure: honor dependencies of dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6603
diff changeset
41
6383
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
42 libs=
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
43 for lib in $ngx_module_libs
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
44 do
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
45 case $lib in
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
46
6419
39a806ccf21e Dynamic modules: perl.
Ruslan Ermilov <ru@nginx.com>
parents: 6383
diff changeset
47 LIBXSLT | LIBGD | GEOIP | PERL)
6383
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
48 libs="$libs \$NGX_LIB_$lib"
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
49
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
50 if eval [ "\$USE_${lib}" = NO ] ; then
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
51 eval USE_${lib}=DYNAMIC
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
52 fi
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
53 ;;
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
54
6603
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
55 PCRE | OPENSSL | ZLIB)
6383
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
56 eval USE_${lib}=YES
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
57 ;;
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
58
6603
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
59 MD5 | SHA1)
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
60 # obsolete
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
61 ;;
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
62
6383
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
63 *)
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
64 libs="$libs $lib"
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
65 ;;
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
66
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
67 esac
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
68 done
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
69 eval ${ngx_module}_LIBS=\'$libs\'
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
70
85dea406e18f Dynamic modules.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6382
diff changeset
71 elif [ "$ngx_module_link" = YES ]; then
6382
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
72
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
73 eval ${ngx_module_type}_MODULES=\"\$${ngx_module_type}_MODULES \
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
74 $ngx_module_name\"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
75
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
76 eval ${ngx_var}_SRCS=\"\$${ngx_var}_SRCS $ngx_module_srcs\"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
77
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
78 if test -n "$ngx_module_incs"; then
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
79 eval ${ngx_var}_INCS=\"\$${ngx_var}_INCS $ngx_module_incs\"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
80 fi
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
81
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
82 if test -n "$ngx_module_deps"; then
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
83 eval ${ngx_var}_DEPS=\"\$${ngx_var}_DEPS $ngx_module_deps\"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
84 fi
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
85
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
86 for lib in $ngx_module_libs
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
87 do
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
88 case $lib in
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
89
6603
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
90 PCRE | OPENSSL | ZLIB | LIBXSLT | LIBGD | PERL | GEOIP)
6382
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
91 eval USE_${lib}=YES
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
92 ;;
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
93
6603
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
94 MD5 | SHA1)
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
95 # obsolete
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
96 ;;
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
97
6382
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
98 *)
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
99 CORE_LIBS="$CORE_LIBS $lib"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
100 ;;
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
101
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
102 esac
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
103 done
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
104
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
105 elif [ "$ngx_module_link" = ADDON ]; then
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
106
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
107 eval ${ngx_module_type}_MODULES=\"\$${ngx_module_type}_MODULES \
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
108 $ngx_module_name\"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
109
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
110 NGX_ADDON_SRCS="$NGX_ADDON_SRCS $ngx_module_srcs"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
111
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
112 if test -n "$ngx_module_incs"; then
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
113 eval ${ngx_var}_INCS=\"\$${ngx_var}_INCS $ngx_module_incs\"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
114 fi
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
115
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
116 if test -n "$ngx_module_deps"; then
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
117 NGX_ADDON_DEPS="$NGX_ADDON_DEPS $ngx_module_deps"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
118 fi
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
119
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
120 for lib in $ngx_module_libs
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
121 do
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
122 case $lib in
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
123
6603
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
124 PCRE | OPENSSL | ZLIB | LIBXSLT | LIBGD | PERL | GEOIP)
6382
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
125 eval USE_${lib}=YES
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
126 ;;
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
127
6603
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
128 MD5 | SHA1)
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
129 # obsolete
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
130 ;;
9eefb38f0005 Internal md5 and sha1 implementations are now always used.
Maxim Dounin <mdounin@mdounin.ru>
parents: 6419
diff changeset
131
6382
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
132 *)
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
133 CORE_LIBS="$CORE_LIBS $lib"
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
134 ;;
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
135
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
136 esac
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
137 done
392959224560 Dynamic modules: auto/module script.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
138 fi