Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Stream counts are confusingly documented #1941

Closed
ianswett opened this issue Oct 31, 2018 · 1 comment
Closed

Stream counts are confusingly documented #1941

ianswett opened this issue Oct 31, 2018 · 1 comment
Assignees
@ianswett
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@ianswett
Copy link
Contributor

#1906 moved from max stream ID to stream counts, but it's insufficiently clear that these streams must still be opened in stream ID order and the limit is across the life of the connection, not just simultaneously open streams.

Would you like me to write a PR or would someone else like to?
@ianswett ianswett added editorial An issue that does not affect the design of the protocol; does not require consensus. -transport labels Oct 31, 2018
@martinthomson
Copy link
Member

You have the itch.

ianswett added a commit that referenced this issue Nov 15, 2018
@ianswett
Clarify MAX_STREAMS
9335f5a 
Fixes #1941
@ianswett ianswett mentioned this issue Nov 15, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ianswett ianswett

Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@martinthomson @ianswett