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

Maxim Dounin mdounin at mdounin.ru
Thu Feb 20 15:55:06 UTC 2020


details:   https://hg.nginx.org/nginx/rev/fe5976aae0e3
branches:  
changeset: 7626:fe5976aae0e3
user:      Maxim Dounin <mdounin at 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);


More information about the nginx-devel mailing list