Welcome! Log In Create A New Profile

Advanced

[nginx] Removed "Transfer-Encoding: identity" support.

Maxim Dounin
February 20, 2020 10:56AM
details: https://hg.nginx.org/nginx/rev/fe5976aae0e3
branches:
changeset: 7626:fe5976aae0e3
user: Maxim Dounin <mdounin@mdounin.ru>
date: Thu Feb 20 16:19:34 2020 +0300
description:
Removed "Transfer-Encoding: identity" support.

The "identity" transfer coding has been removed in RFC 7230. It is
believed that it is not used in real life, and at the same time it
provides a potential attack vector.

diffstat:

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

diffs (15 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
@@ -1952,10 +1952,7 @@ ngx_http_process_request_header(ngx_http
r->headers_in.content_length_n = -1;
r->headers_in.chunked = 1;

- } else if (r->headers_in.transfer_encoding->value.len != 8
- || ngx_strncasecmp(r->headers_in.transfer_encoding->value.data,
- (u_char *) "identity", 8) != 0)
- {
+ } else {
ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
"client sent unknown \"Transfer-Encoding\": \"%V\"",
&r->headers_in.transfer_encoding->value);
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[nginx] Removed "Transfer-Encoding: identity" support.

Maxim Dounin 71 February 20, 2020 10:56AM



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

Online Users

Guests: 79
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