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

State that transfer codings should not use parameters named "q" (closes #15) #109

Merged
merged 1 commit into from
Jun 29, 2018

Conversation

reschke
Copy link
Contributor

@reschke reschke commented Jun 26, 2018

No description provided.

@reschke reschke requested review from mnot and royfielding June 26, 2018 17:38
@reschke reschke changed the title State that transfer codings should not use parameters named "q" (clos… State that transfer codings should not use parameters named "q" (closes #15) Jun 26, 2018
@royfielding
Copy link
Member

That needs some tweaks ... the action words like SHOULD are supposed to be targeted to implementation roles, not protocol extensions. Can we just copy the same Note from the Accept section?

@reschke
Copy link
Contributor Author

reschke commented Jun 26, 2018

I can avoid the SHOULD NOT.

I used different text because for transfer codings "we" actually own the registry, while for content types we do not...

@royfielding
Copy link
Member

Heh, I think it's a stretch of the imagination to think that we control the registry even when we define it ... people still deploy first and register later. In any case, a simple

If possible, avoid defining transfer codings with parameters; if you have no choice, don't use "q" as a parameter name because it will be confused with the quality ranking parameter of that name.

should be sufficient.

@reschke
Copy link
Contributor Author

reschke commented Jun 27, 2018

I actually reverse my opinion about the SHOULD NOT - we already use BCP14 keywords in the paragraphs before and after this one. So please consider again the pull request.

@mnot
Copy link
Member

mnot commented Jun 27, 2018

My .02 - acknowledging that 2119 keywords are only intended for implementations roles, I don't think that throwing up our hands and saying "people will do what they will" is productive.

While some people ignore (or just don't read) the RFCs, many do, and when they do, we owe it to them to communicate in the clearest, concise way possible. 2119 keywords are a fantastic tool for doing that.

@reschke
Copy link
Contributor Author

reschke commented Jun 28, 2018

@mnot - so are you ok with this change?

@mnot
Copy link
Member

mnot commented Jun 29, 2018

I am - although if we come up with a unified way to deal with requirements consistently across the documents, it might change.

@reschke reschke merged commit a802c49 into master Jun 29, 2018
@reschke reschke deleted the reschke-eid4683 branch February 17, 2019 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants