From: Roman Arutyunyan <arut@nginx.com>
Date: Fri, 29 Jan 2021 16:42:47 +0000 (+0300)
Subject: HTTP/3: call ngx_handle_read_event() from client header handler.
X-Git-Url: http://git.kaiwu.me/postgresql/log/contrib/postgres_fdw/static/gitweb.js?a=commitdiff_plain;h=cd6253430051a823dc31b756e93aeecb5f674af3;p=nginx.git

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.
---

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) {