shinrich commented on a change in pull request #6889:
URL: https://github.com/apache/trafficserver/pull/6889#discussion_r452256385



##########
File path: proxy/http2/Http2ConnectionState.cc
##########
@@ -242,7 +242,7 @@ rcv_headers_frame(Http2ConnectionState &cstate, const 
Http2Frame &frame)
   if (cstate.is_valid_streamid(stream_id)) {
     stream = cstate.find_stream(stream_id);
     if (stream == nullptr || !stream->has_trailing_header()) {
-      return Http2Error(Http2ErrorClass::HTTP2_ERROR_CLASS_CONNECTION, 
Http2ErrorCode::HTTP2_ERROR_STREAM_CLOSED,

Review comment:
       This was to fix one of the h2spec cases.  I'll update to split the cases 
here to return two different error codes.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to