[nginx] HTTP/2: avoid adding Content-Length for requests without body.
Maxim Dounin
mdounin at mdounin.ru
Tue Oct 18 14:48:03 UTC 2016
details: http://hg.nginx.org/nginx/rev/12b1df4c9944
branches: stable-1.10
changeset: 6754:12b1df4c9944
user: Valentin Bartenev <vbart at nginx.com>
date: Thu Jun 16 20:55:11 2016 +0300
description:
HTTP/2: avoid adding Content-Length for requests without body.
There is no reason to add the "Content-Length: 0" header to a proxied request
without body if the header isn't presented in the original request.
Thanks to Amazon.
diffstat:
src/http/v2/ngx_http_v2.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diffs (23 lines):
diff --git a/src/http/v2/ngx_http_v2.c b/src/http/v2/ngx_http_v2.c
--- a/src/http/v2/ngx_http_v2.c
+++ b/src/http/v2/ngx_http_v2.c
@@ -3424,7 +3424,9 @@ ngx_http_v2_run_request(ngx_http_request
return;
}
- r->headers_in.chunked = (r->headers_in.content_length_n == -1);
+ if (r->headers_in.content_length_n == -1 && !r->stream->in_closed) {
+ r->headers_in.chunked = 1;
+ }
ngx_http_process_request(r);
}
@@ -3638,7 +3640,7 @@ ngx_http_v2_process_request_body(ngx_htt
rb->buf = NULL;
}
- if (r->headers_in.content_length_n == -1) {
+ if (r->headers_in.chunked) {
r->headers_in.content_length_n = rb->received;
}
More information about the nginx-devel
mailing list