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

Defaulting to no-push via SETTINGS_MAX_CONCURRENT_STREAMS #40

Closed
OsamaM opened this issue Feb 21, 2013 · 3 comments
Closed

Defaulting to no-push via SETTINGS_MAX_CONCURRENT_STREAMS #40

OsamaM opened this issue Feb 21, 2013 · 3 comments

Comments

@OsamaM
Copy link

@OsamaM OsamaM commented Feb 21, 2013

This is fork of issue #38 to focus on discussing "defaulting to no-push."

The thinking is that client and server endpoints should have different minimum values and default values for SETTINGS_MAX_CONCURRENT_STREAMS.

From issue #20 we understand that SETTINGS_MAX_CONCURRENT_STREAMS is directional and a client can advertise a value of 0 to prevent the server from issuing Push Streams.

The suggestion is that the default value should also be 0. That is, the communication defaults to no-push. If the client is sophisticated then it can advertise a non-zero value in its initial SETTINGS frame to allow the server to push.

@mnot
Copy link
Member

@mnot mnot commented Jun 13, 2013

DIscussed in SF Interim; decided to wait until we have more implementation experience.

@martinthomson
Copy link
Collaborator

@martinthomson martinthomson commented Jun 26, 2013

With the changes to include HTTP2-Settings, the server never sends frames in ignorance of the clients settings. Rather than deferring, can we close this @OsamaM ?

@OsamaM OsamaM closed this Jul 20, 2013
@OsamaM
Copy link
Author

@OsamaM OsamaM commented Jul 20, 2013

Solved via "HTTP2-Settings Header Field" section.

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