Mercurial > hg > nginx

changeset 7909:f302c1096f7b

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
HTTP/2: improved handling of END_STREAM in a separate DATA frame. The save body filter saves the request body to disk once the buffer is full. Yet in HTTP/2 this might happen even if there is no need to save anything to disk, notably when content length is known and the END_STREAM flag is sent in a separate empty DATA frame. Workaround is to provide additional byte in the buffer, so saving the request body won't be triggered. This fixes unexpected request body disk buffering in HTTP/2 observed after the previous change when content length is known and the END_STREAM flag is sent in a separate empty DATA frame.
author Maxim Dounin <mdounin@mdounin.ru>
date Sun, 29 Aug 2021 22:20:38 +0300
parents 0dcec8e5d50a
children 1d78437dbc3f
files src/http/v2/ngx_http_v2.c
diffstat 1 files changed, 3 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/http/v2/ngx_http_v2.c	Sun Aug 29 22:20:36 2021 +0300
+++ b/src/http/v2/ngx_http_v2.c	Sun Aug 29 22:20:38 2021 +0300
@@ -4034,6 +4034,9 @@
 
     if (len < 0 || len > (off_t) clcf->client_body_buffer_size) {
         len = clcf->client_body_buffer_size;
+
+    } else {
+        len++;
     }
 
     if (r->request_body_no_buffering && !stream->in_closed) {