From cd6253430051a823dc31b756e93aeecb5f674af3 Mon Sep 17 00:00:00 2001 From: Roman Arutyunyan Date: Fri, 29 Jan 2021 19:42:47 +0300 Subject: [PATCH] HTTP/3: call ngx_handle_read_event() from client header handler. This function should be called at the end of an event handler to prepare the event for the next handler call. Particularly, the "active" flag is set or cleared depending on data availability. With this call missing in one code path, read handler was not called again after handling the initial part of the client request, if the request was too big to fit into a single STREAM frame. Now ngx_handle_read_event() is called in this code path. Also, read timer is restarted. --- src/http/v3/ngx_http_v3_request.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/http/v3/ngx_http_v3_request.c b/src/http/v3/ngx_http_v3_request.c index fb1626718..0b7954137 100644 --- a/src/http/v3/ngx_http_v3_request.c +++ b/src/http/v3/ngx_http_v3_request.c @@ -158,11 +158,12 @@ ngx_http_v3_process_request(ngx_event_t *rev) if (b->pos == b->last) { - if (!rev->ready) { - break; - } + if (rev->ready) { + n = c->recv(c, b->start, b->end - b->start); - n = c->recv(c, b->start, b->end - b->start); + } else { + n = NGX_AGAIN; + } if (n == NGX_AGAIN) { if (!rev->timer_set) {