8

http2: update handling of streams on rst_stream frames by kumarak · Pull Request...

 2 years ago
source link: https://github.com/nodejs/node/pull/39622
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

Copy link

Member

kumarak commented on Aug 2, 2021

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: #39423


About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK