nginx: auto/endianness annotate

annotate auto/endianness @ 7746:88eca63261c3

gRPC: RST_STREAM(NO_ERROR) handling after "trailer only" responses. Similarly to the problem fixed in 2096b21fcd10 (ticket #1792), when a "trailer only" gRPC response (that is, a response with the END_STREAM flag in the HEADERS frame) was immediately followed by RST_STREAM(NO_ERROR) in the data preread along with the response header, RST_STREAM wasn't properly skipped and caused "upstream rejected request with error 0" errors. Observed with "unknown service" gRPC errors returned by grpc-go. Fix is to set ctx->done if we are going to parse additional data, so the RST_STREAM(NO_ERROR) is properly skipped. Additionally, now ngx_http_grpc_filter() will complain about frames sent for closed stream if there are any.

author	Pavel Pautov <p.pautov@f5.com>
date	Wed, 18 Nov 2020 18:41:16 -0800
parents	e3faa5fb7772
children

rev	line source
444 42d11f017717 nginx-0.1.0-2004-09-29-20:00:49 import; remove years from copyright Igor Sysoev <igor@sysoev.ru> parents: 370 diff changeset	1
42d11f017717 nginx-0.1.0-2004-09-29-20:00:49 import; remove years from copyright Igor Sysoev <igor@sysoev.ru> parents: 370 diff changeset	2 # Copyright (C) Igor Sysoev
4412 d620f497c50f Copyright updated. Maxim Konovalov <maxim@nginx.com> parents: 703 diff changeset	3 # Copyright (C) Nginx, Inc.
444 42d11f017717 nginx-0.1.0-2004-09-29-20:00:49 import; remove years from copyright Igor Sysoev <igor@sysoev.ru> parents: 370 diff changeset	4
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	5
4681 bb37a9cc08fb Fixed spelling of "endianness", and called it "byte ordering" in the Ruslan Ermilov <ru@nginx.com> parents: 4412 diff changeset	6 echo $ngx_n "checking for system byte ordering ...$ngx_c"
6269 7ec809b579d7 Configure: style fixes for autoconf.err. Piotr Sikora <piotrsikora@google.com> parents: 5309 diff changeset	7
7ec809b579d7 Configure: style fixes for autoconf.err. Piotr Sikora <piotrsikora@google.com> parents: 5309 diff changeset	8 cat << END >> $NGX_AUTOCONF_ERR
7ec809b579d7 Configure: style fixes for autoconf.err. Piotr Sikora <piotrsikora@google.com> parents: 5309 diff changeset	9
7ec809b579d7 Configure: style fixes for autoconf.err. Piotr Sikora <piotrsikora@google.com> parents: 5309 diff changeset	10 ----------------------------------------
7ec809b579d7 Configure: style fixes for autoconf.err. Piotr Sikora <piotrsikora@google.com> parents: 5309 diff changeset	11 checking for system byte ordering
7ec809b579d7 Configure: style fixes for autoconf.err. Piotr Sikora <piotrsikora@google.com> parents: 5309 diff changeset	12
7ec809b579d7 Configure: style fixes for autoconf.err. Piotr Sikora <piotrsikora@google.com> parents: 5309 diff changeset	13 END
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	14
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	15
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	16 cat << END > $NGX_AUTOTEST.c
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	17
6624 e3faa5fb7772 Configure: fix build with -Werror=old-style-definition. Piotr Sikora <piotrsikora@google.com> parents: 6269 diff changeset	18 int main(void) {
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	19 int i = 0x11223344;
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	20 char *p;
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	21
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	22 p = (char *) &i;
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	23 if (*p == 0x44) return 0;
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	24 return 1;
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	25 }
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	26
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	27 END
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	28
583 4e296b7d25bf nginx-0.3.13-RELEASE import Igor Sysoev <igor@sysoev.ru> parents: 577 diff changeset	29 ngx_test="$CC $CC_TEST_FLAGS $CC_AUX_FLAGS \
577 4d9ea73a627a nginx-0.3.10-RELEASE import Igor Sysoev <igor@sysoev.ru> parents: 469 diff changeset	30 -o $NGX_AUTOTEST $NGX_AUTOTEST.c $NGX_LD_OPT $ngx_feature_libs"
4d9ea73a627a nginx-0.3.10-RELEASE import Igor Sysoev <igor@sysoev.ru> parents: 469 diff changeset	31
4d9ea73a627a nginx-0.3.10-RELEASE import Igor Sysoev <igor@sysoev.ru> parents: 469 diff changeset	32 eval "$ngx_test >> $NGX_AUTOCONF_ERR 2>&1"
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	33
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	34 if [ -x $NGX_AUTOTEST ]; then
703 694d285ec752 fix ">/dev/null 2>&1" order Igor Sysoev <igor@sysoev.ru> parents: 583 diff changeset	35 if $NGX_AUTOTEST >/dev/null 2>&1; then
4681 bb37a9cc08fb Fixed spelling of "endianness", and called it "byte ordering" in the Ruslan Ermilov <ru@nginx.com> parents: 4412 diff changeset	36 echo " little endian"
469 2ff194b74f1e nginx-0.1.9-RELEASE import Igor Sysoev <igor@sysoev.ru> parents: 455 diff changeset	37 have=NGX_HAVE_LITTLE_ENDIAN . auto/have
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	38 else
4681 bb37a9cc08fb Fixed spelling of "endianness", and called it "byte ordering" in the Ruslan Ermilov <ru@nginx.com> parents: 4412 diff changeset	39 echo " big endian"
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	40 fi
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	41
5309 434548349838 Configure: fixed autotest cleanup commands. Sergey Kandaurov <pluknet@nginx.com> parents: 4681 diff changeset	42 rm -rf $NGX_AUTOTEST*
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	43
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	44 else
5309 434548349838 Configure: fixed autotest cleanup commands. Sergey Kandaurov <pluknet@nginx.com> parents: 4681 diff changeset	45 rm -rf $NGX_AUTOTEST*
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	46
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	47 echo
4681 bb37a9cc08fb Fixed spelling of "endianness", and called it "byte ordering" in the Ruslan Ermilov <ru@nginx.com> parents: 4412 diff changeset	48 echo "$0: error: cannot detect system byte ordering"
370 54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	49 exit 1
54f76b0b8dca nginx-0.0.7-2004-06-27-22:01:57 import Igor Sysoev <igor@sysoev.ru> parents: diff changeset	50 fi

Mercurial > hg > nginx

annotate auto/endianness @ 7746:88eca63261c3