Mercurial > hg > nginx
comparison src/http/v2/ngx_http_v2.c @ 7909:f302c1096f7b
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 |
comparison
equal
deleted
inserted
replaced
| 7908:0dcec8e5d50a | 7909:f302c1096f7b |
|---|---|
| 4032 | 4032 |
| 4033 len = r->headers_in.content_length_n; | 4033 len = r->headers_in.content_length_n; |
| 4034 | 4034 |
| 4035 if (len < 0 || len > (off_t) clcf->client_body_buffer_size) { | 4035 if (len < 0 || len > (off_t) clcf->client_body_buffer_size) { |
| 4036 len = clcf->client_body_buffer_size; | 4036 len = clcf->client_body_buffer_size; |
| 4037 | |
| 4038 } else { | |
| 4039 len++; | |
| 4037 } | 4040 } |
| 4038 | 4041 |
| 4039 if (r->request_body_no_buffering && !stream->in_closed) { | 4042 if (r->request_body_no_buffering && !stream->in_closed) { |
| 4040 | 4043 |
| 4041 /* | 4044 /* |