]> git.kaiwu.me - nginx.git/commit
HTTP/2: implemented preread buffer for request body (closes #959).
authorValentin Bartenev <vbart@nginx.com>
Tue, 24 May 2016 14:37:52 +0000 (17:37 +0300)
committerValentin Bartenev <vbart@nginx.com>
Tue, 24 May 2016 14:37:52 +0000 (17:37 +0300)
commit5429140c018726e8b28c724b06daaf0fe33d2205
treea4472f68dd62a8f30acc7df95a450e5e9515b861
parentadfd0b065cca23d2ca61ad43c5b8fdaa23957da9
HTTP/2: implemented preread buffer for request body (closes #959).

Previously, the stream's window was kept zero in order to prevent a client
from sending the request body before it was requested (see 887cca40ba6a for
details).  Until such initial window was acknowledged all requests with
data were rejected (see 0aa07850922f for details).

That approach revealed a number of problems:

 1. Some clients (notably MS IE/Edge, Safari, iOS applications) show an error
    or even crash if a stream is rejected;

 2. This requires at least one RTT for every request with body before the
    client receives window update and able to send data.

To overcome these problems the new directive "http2_body_preread_size" is
introduced.  It sets the initial window and configures a special per stream
preread buffer that is used to save all incoming data before the body is
requested and processed.

If the directive's value is lower than the default initial window (65535),
as previously, all streams with data will be rejected until the new window
is acknowledged.  Otherwise, no special processing is used and all requests
with data are welcome right from the connection start.

The default value is chosen to be 64k, which is bigger than the default
initial window.  Setting it to zero is fully complaint to the previous
behavior.
src/http/v2/ngx_http_v2.c
src/http/v2/ngx_http_v2.h
src/http/v2/ngx_http_v2_module.c
src/http/v2/ngx_http_v2_module.h