tree 2cb0cedb7e44259e9a1e2059bebd484f3fb5d43d
parent a7d5f74ddabe321c89dbe73ab11f4be6cf88d0d9
author QUICHE team <quiche-dev@google.com> 1638824199 -0800
committer Copybara-Service <copybara-worker@google.com> 1638824277 -0800

Enable OgHttp2Session to RST_STREAM overflow streams with unacknowledged MAX_CONCURRENT_STREAMS values.

This CL adds a pending max inbound concurrent streams field to OgHttp2Session,
which stores the pending MAX_CONCURRENT_STREAMS value sent by OgHttp2Session
but not yet acknowledged by the peer. Then OgHttp2Session can reject incoming
streams exceeding the value with a RST_STREAM REFUSED_STREAM (not GOAWAY),
which better aligns oghttp2 with nghttp2 behavior.

This change affects at least two codec_impl_test tests:
http://sponge2/d291d8b8-2281-48b2-ae8f-203c94b0f484

If test mutual dispatch resolution is merged
(https://github.com/envoyproxy/envoy/pull/19195),
Http2CodecImplStreamLimitTest.LazyDecreaseMaxConcurrentStreamsConsumeError
passes, and
Http2CodecImplStreamLimitTest.LazyDecreaseMaxConcurrentStreamsIgnoreError
passes mod some nghttp2-specific error message [*].

[*] http://google3/third_party/envoy/src/source/common/http/http2/codec_impl.cc;l=850;rcl=410881808

PiperOrigin-RevId: 414519202