tree 14a773ac7707ac4581c8ea11ca2b3a42f1ac88f8
parent 7491b0e3bae338d61809c1f9e4ddd1225104fd75
author QUICHE team <quiche-dev@google.com> 1642613578 -0800
committer Copybara-Service <copybara-worker@google.com> 1642613658 -0800

Avoid OnInvalidFrame() to make content-length errors stream-level in oghttp2.

As background, cl/422597021 made oghttp2 follow nghttp2 on calling the visitor
OnInvalidFrame() to make an integration test pass with a connection-level error
expectation [1], but another test expects a stream-level error instead [2].

These tests both pass with nghttp2 because nghttp2 handles content-length
differently based on whether the end_stream frame is headers/trailers vs.  data
(see cl/422597021 description for details). However, preferred behavior is for
content-length errors to be stream-level across the board.

This CL updates oghttp2 such that content-length errors do not call
OnInvalidFrame() and are thus stream-level, for both tests. The expectations in
[1] will be changed to align with HTTP/3.

[1]
http://google3/third_party/envoy/src/test/integration/multiplexed_integration_test.cc;l=1972;rcl=414009764
[2]
http://google3/third_party/envoy/src/test/integration/protocol_integration_test.cc;l=3535;rcl=421694486

PiperOrigin-RevId: 422833845