Skip to content

http2: update handling of streams on rst_stream frames

Rodrigo Muino Tomonari requested to merge github/fork/kumarak/kumarak/handling_rst_stream into master

The PR updates the handling of rst_stream frames and add all streams to the pending list on receiving rst frames with the error code NGHTTP2_CANCEL.

The changes will remove dependency on the stream state that may allow bypassing the checks in certain cases. I think a better solution is to delay streams in all cases if rst_stream is received for the cancel events.

The rst_stream frames can be received for protocol/connection error as well it should be handled immediately. Adding streams to the pending list in such cases may cause errors.

Previous PR ref: https://github.com/nodejs/node/pull/39423

Merge request reports