Skip to content
This repository was archived by the owner on Nov 24, 2019. It is now read-only.

Commit 0b599f4

Browse files
adamhenson75lb
authored andcommitted
Fixes molnarg#107: Allow frames after RST_STREAM.
Prevent exceptions for the common case in which the serving endpoint sends frames after the RST_STREAM has been sent from the client endpoint. Often frames are enqueued prior to the arrival of the RST_STREAM, but sent after. This should be expected based on HTTP/2 spec, section 6.4. https://http2.github.io/http2-spec/#rfc.section.6.4 "However, after sending the RST_STREAM, the sending endpoint MUST be prepared to receive and process additional frames sent on the stream that might have been sent by the peer prior to the arrival of the RST_STREAM."
1 parent 7348b03 commit 0b599f4

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

lib/protocol/stream.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -567,6 +567,7 @@ Stream.prototype._transition = function transition(sending, frame) {
567567
// can be used to close any of those streams.
568568
case 'CLOSED':
569569
if (PRIORITY || (sending && RST_STREAM) ||
570+
(sending && this._closedWithRst) ||
570571
(receiving && WINDOW_UPDATE) ||
571572
(receiving && this._closedByUs &&
572573
(this._closedWithRst || RST_STREAM || ALTSVC))) {

0 commit comments

Comments
 (0)