maskit opened a new pull request, #13387:
URL: https://github.com/apache/trafficserver/pull/13387
proxy.config.http2.max_active_streams_in has only adjusted the advertised
SETTINGS_MAX_CONCURRENT_STREAMS, leaving the proxy unable to bound buffered
response memory against clients that open streams faster than the advisory
throttle reacts. It now also carries a finite default so the cap is no longer
effectively unlimited.
The new knob proxy.config.http2.max_active_streams_policy_in selects
enforcement. Value 0 keeps the advisory behavior that lowers advertised
concurrency to proxy.config.http2.min_concurrent_streams_in. Value 1 refuses
new inbound streams with REFUSED_STREAM once the process-wide active-stream
count reaches the limit and leaves the advertised value untouched, so the
min_concurrent_streams_in reduction that disrupts some clients no longer
applies.
max_active_streams_in now defaults to 200000 rather than 0. Operators should
size it to their available memory budget; set it to 0 to disable the cap.
Refusal happens after HPACK decoding so the dynamic table stays in sync with
the client and the connection survives.
proxy.process.http2.max_active_streams_exceeded_in counts each refusal.
(cherry picked from commit c76a7c81220fca009ff4dc7de91647c3cf6d72d7)
Conflicts:
src/proxy/http2/HTTP2.cc
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
