bryancall commented on code in PR #13351:
URL: https://github.com/apache/trafficserver/pull/13351#discussion_r3571999416
##########
src/proxy/http2/Http2ConnectionState.cc:
##########
@@ -510,6 +545,15 @@ Http2ConnectionState::rcv_headers_frame(const Http2Frame
&frame)
"recv data bad payload length");
}
+ // Discard an interim (1xx) response from the
+ // origin and wait for the final response on this stream.
+ if (is_outbound_interim_response(stream)) {
Review Comment:
Fixed in 727bb7a4a3: ATS now returns a stream PROTOCOL_ERROR when a 1xx
interim response sets END_STREAM (RFC 9113 8.1), on both the HEADERS and
CONTINUATION paths, and I added a gold test case covering it.
_🤖 Addressed by [Claude Code](https://claude.com/claude-code)_
##########
src/proxy/http2/Http2ConnectionState.cc:
##########
@@ -1125,6 +1178,15 @@ Http2ConnectionState::rcv_continuation_frame(const
Http2Frame &frame)
"recv data bad payload length");
}
+ // Discard an interim (1xx) response from the
+ // origin and wait for the final response on this stream.
+ if (is_outbound_interim_response(stream)) {
Review Comment:
Fixed in 727bb7a4a3: ATS now returns a stream PROTOCOL_ERROR when a 1xx
interim response sets END_STREAM (RFC 9113 8.1), on both the HEADERS and
CONTINUATION paths, and I added a gold test case covering it.
_🤖 Addressed by [Claude Code](https://claude.com/claude-code)_
--
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]
