Welcome! Log In Create A New Profile


[nginx] Disabled multiple Transfer-Encoding headers.

Maxim Dounin
February 20, 2020 10:56AM
details: https://hg.nginx.org/nginx/rev/aca005d232ff
changeset: 7625:aca005d232ff
user: Maxim Dounin <mdounin@mdounin.ru>
date: Thu Feb 20 16:19:29 2020 +0300
Disabled multiple Transfer-Encoding headers.

We anyway do not support more than one transfer encoding, so accepting
requests with multiple Transfer-Encoding headers doesn't make sense.
Further, we do not handle multiple headers, and ignore anything but
the first header.

Reported by Filippo Valsorda.


src/http/ngx_http_request.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff --git a/src/http/ngx_http_request.c b/src/http/ngx_http_request.c
--- a/src/http/ngx_http_request.c
+++ b/src/http/ngx_http_request.c
@@ -131,7 +131,7 @@ ngx_http_header_t ngx_http_headers_in[]

{ ngx_string("Transfer-Encoding"),
offsetof(ngx_http_headers_in_t, transfer_encoding),
- ngx_http_process_header_line },
+ ngx_http_process_unique_header_line },

{ ngx_string("TE"),
offsetof(ngx_http_headers_in_t, te),
nginx-devel mailing list
Subject Author Views Posted

[nginx] Disabled multiple Transfer-Encoding headers.

Maxim Dounin 71 February 20, 2020 10:56AM

Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 83
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready