commit 2939d5c5fadb3f28a8f8ac85c039bfd007239d3c
author Piotr Sikora <piotrsikora@google.com> Wed Oct 24 19:47:16 2018 -0700
committer Piotr Sikora <piotrsikora@google.com> Thu Oct 25 18:24:30 2018 +0000
tree bc4a1f13e248f885afbd7832d9abf3ee4eec9304
parent ef5418c52f115a85492dbc6b93e743669c56de1a

HTTP/2: fixed handling of fully preread request bodies.

Previously, fully preread request body of a request without the
"Content-Length" header was always written to a temporary file.

Change-Id: I996d1be9aca1b7ec6b0c139111150d0b832abb12
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
Reviewed-on: https://nginx-review.googlesource.com/c/3540
Reviewed-by: Wayne Zhang <qiwzhang@google.com>

src/http/v2/ngx_http_v2.c

diff --git a/src/http/v2/ngx_http_v2.c b/src/http/v2/ngx_http_v2.c
index 80520ee..04b3385 100644
--- a/src/http/v2/ngx_http_v2.c
+++ b/src/http/v2/ngx_http_v2.c
@@ -3896,6 +3896,12 @@
     {
         rb->buf = ngx_create_temp_buf(r->pool, (size_t) len);
 
+    } else if (len < 0 && stream->in_closed && stream->preread
+               && !r->request_body_in_file_only)
+    {
+        rb->buf = ngx_create_temp_buf(r->pool,
+                                      (size_t) ngx_buf_size(stream->preread));
+
     } else {
         rb->buf = ngx_calloc_buf(r->pool);