/[Apache-SVN]
ViewVC logotype

Revision 1788146


Jump to revision: Previous Next
Author: kotkov
Date: Wed Mar 22 18:07:51 2017 UTC (7 years, 2 months ago)
Changed paths: 1
Log Message:
HTTP/2: Fix improper handling of SETTINGS_INITIAL_WINDOW_SIZE that
could result in the "HTTP2 flow control limits exceeded" error when
talking to nginx servers:

    > serf_get --http2 https://www.cloudflare.com
    Error running context: (120153) HTTP2 flow control limits exceeded

Serf used the incoming value to update the size of the connection
flow-control window.  Doing so violates RFC 7540, 6.9.2 [1], which
states that the SETTINGS_INITIAL_WINDOW_SIZE value in the SETTINGS
frame *cannot* alter the connection flow-control window size, and
that it only affects the initial window size for new streams.

[1] https://tools.ietf.org/html/rfc7540#section-6.9.2

* protocols/http2_protocol.c
  (http2_handle_settings): Don't change the connection flow-control
   window size.  Include the relevant part of RFC 7540 in the comment.

Approved by: rhuijben


Changed paths

Path Details
Directoryserf/trunk/protocols/http2_protocol.c modified , text changed

infrastructure at apache.org
ViewVC Help
Powered by ViewVC 1.1.26