Skip to content

Allow receipt of WINDOW_UPDATE on closed streams#172

Merged
nwgh merged 1 commit into
molnarg:masterfrom
tbetbetbe:bugfix-explicitly-permit-window-updates-receipt-on-closed-streams
Jan 28, 2016
Merged

Allow receipt of WINDOW_UPDATE on closed streams#172
nwgh merged 1 commit into
molnarg:masterfrom
tbetbetbe:bugfix-explicitly-permit-window-updates-receipt-on-closed-streams

Conversation

@tbetbetbe
Copy link
Copy Markdown
Contributor

https://httpwg.github.io/specs/rfc7540.html#StreamStates

'WINDOW_UPDATE or RST_STREAM frames can be received in this state for a
short period after a DATA or HEADERS frame containing an END_STREAM flag
is sent.'

The current logic does not allow this work in all valid cases,

https://httpwg.github.io/specs/rfc7540.html#StreamStates

'WINDOW_UPDATE or RST_STREAM frames can be received in this state for a
short period after a DATA or HEADERS frame containing an END_STREAM flag
is sent.'
nwgh added a commit that referenced this pull request Jan 28, 2016
…w-updates-receipt-on-closed-streams

Allow receipt of WINDOW_UPDATE on closed streams
@nwgh nwgh merged commit 8570a15 into molnarg:master Jan 28, 2016
@nwgh
Copy link
Copy Markdown
Collaborator

nwgh commented Jan 28, 2016

Thanks!

tbetbetbe pushed a commit to tbetbetbe/node-http2 that referenced this pull request Jan 28, 2016
This covers the contributions in PRs molnarg#170, molnarg#171 and molnarg#172
mcmanus pushed a commit to mcmanus/node-http2 that referenced this pull request Apr 3, 2017
This covers the contributions in PRs molnarg#170, molnarg#171 and molnarg#172
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants