]> 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)
commit323c6ba0b7bca5174f1f717895b5a52e5af4bd86
tree275efe3e90a50f996d271909d6818285ff19b266
parentf0c3ae9fc37b21e42b9430554391b17b79a70000
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