tree 847c7f833bc489446d07708755801a234399c6fc
parent d5b9d6280163efe26b4f5a1a1c83f8f5211d44c7
author diannahu <diannahu@google.com> 1649182465 -0700
committer Copybara-Service <copybara-worker@google.com> 1649182571 -0700

Add oghttp2 and nghttp2 adapter tests for receiving a GOAWAY with active streams.

This CL introduces adapter tests for the following scenario, inspired directly
by the Envoy test added in [1]:
  1. Client opens two streams, stream IDs 1 and 3.
  2. Server sends RST_STREAM for stream 1.
  3. Server sends GOAWAY with last_stream_id 1.
  4. Server sends WINDOW_UPDATE.
  5. Client should close stream 3.

When receiving a GOAWAY, both nghttp2 [2] and GFE2 [3] close the streams above
the last_stream_id of the GOAWAY. This is a feature gap that oghttp2 needs to
fill. The added tests also show that oghttp2 should avoid invoking the visitor
OnWindowUpdate() for a closed stream (though delivering the common header is
okay/consistent with (o|n)http2 behavior for other frames on closed streams).

[1] http://go/envoy-upstream/pull/20576/files#diff-671fe996ff6537911528a7f54c931b68b93aa199411740ff407b1116ac96a43b
[2] http://google3/third_party/nghttp2/src/lib/nghttp2_session.c;l=4750;rcl=433094660
[3] http://google3/gfe/gfe2/http2/http2_backend_dispatcher.cc;l=642-659;rcl=438051287

PiperOrigin-RevId: 439628963