masaori335 opened a new pull request #7831:
URL: https://github.com/apache/trafficserver/pull/7831


   Prior to the change, `Http2Stream::change_state()` assumes the END_STREAM 
flag comes from the DATA frame on the OPEN state. But HEADERS frame can send 
END_STREAM flag too[*1]. The trailer header case is exactly the case. ( An 
example is on RFC 7540 Section 8.1.3.[*2] ) When Http2Stream falls in this bug, 
it stalls because the stream is not restarted on receiving WINDOW_UPDATE frames.
   
   I haven't noticed this bug with the current AuTest, because the response was 
smaller than the initial window.
   
   [*1] 5.1. Stream States
   https://datatracker.ietf.org/doc/html/rfc7540#section-5.1
   
   [*2] 8.1.3.  Examples (trailing header fields)
   https://datatracker.ietf.org/doc/html/rfc7540#page-59


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