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.

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

Concurrent Streams Limits and Unidirectional Streams #78

Closed
jasnell opened this issue Apr 25, 2013 · 4 comments
Closed

Concurrent Streams Limits and Unidirectional Streams #78

jasnell opened this issue Apr 25, 2013 · 4 comments

Comments

@jasnell
Copy link
Contributor

jasnell commented Apr 25, 2013

In the current draft (-02), we state that:

A. Any endpoint can initiate and half-close a fully unidirectional stream that does not require any action or acknowledgement from the receiving peer. Once half-closed, these remain in a constant half-closed state with the receiving peer having the option of sending frames for that stream at any time for the full duration of the session.

B. An endpoint MUST NOT exceed the maximum concurrent streams limit set by it's peer and that half-closed streams in any direction count towards this limit.

C. Unless I missed it somewhere, clients are not required to half-close PUSH_PROMISE streams.

The potential problem here is simple:

If a client sets a limit of 4 concurrent streams, and the server initiates 4 separate PUSH_PROMISE streams that the server half-closes but that are never half-closed by the client, the server not be able to initiate new push streams for the duration of the session.

@martinthomson
Copy link
Collaborator

On the list...

@jasnell
Copy link
Contributor Author

jasnell commented May 6, 2013

@mnot
Copy link
Member

mnot commented Jun 13, 2013

DIscussed at SF Interim; pending proposal from Layering TF.

@martinthomson
Copy link
Collaborator

This should be addressed by 05df79d

Pending approval of the changes included in the layering branch, this should be ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants