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

MaxNumConcurrentStreams: Update proposed clarification text in all documents. #32

Merged
merged 1 commit into from
Feb 19, 2013
Merged

MaxNumConcurrentStreams: Update proposed clarification text in all documents. #32

merged 1 commit into from
Feb 19, 2013

Conversation

willchan
Copy link
Contributor

This updates the concurrent streams in the XML and updates the HTML too (which picked up some other changes).

@mnot
Copy link
Member

mnot commented Feb 19, 2013

"For implementors it is recommended that this value be no smaller than 100" - what is the significance of "for implementers" here?

I'm concerned this language is ambiguous enough to cause trouble. E.g., some may read it as pure advice; others might take away that values smaller than 100 can be assumed to be 100. Best way to specify is with MUST/SHOULD/MAY, identifying who it applies to (sender / recipient), and ideally specifying what should happen on both ends of the wire (when generating and receiving).

@willchan
Copy link
Contributor Author

I agree this is ambiguous. I snuck in a short explanation there. I feel like we should take this back to the working group mailing list. I propose we either commit as is, as my extra explanatory clause is innocuous, and open an issue to resolve the ambiguity. Or I can withdraw this pull request for now, go back to the mailing list, and proceed with the flow control pull request first.

@mnot
Copy link
Member

mnot commented Feb 19, 2013

I'm happy to merge now and note an issue; just trying to minimise the number of such edits we cause with each change, as they have a tendency to stack up.

mnot added a commit that referenced this pull request Feb 19, 2013
MaxNumConcurrentStreams: Update proposed clarification text in all documents. Fixes #19.
@mnot mnot merged commit c2c03cc into httpwg:master Feb 19, 2013
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