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

Avoid ambiguity with "may not" and a "MAY" requirement #66

Closed
wants to merge 1 commit into from

Conversation

aprescott
Copy link

RFC 2119 defines "MAY" as a keyword, which may cause confusion with "may not", even though the latter is not uppercased.

This hopefully better clarifies that the server in question may or may not be an origin server, as opposed to implying that origin servers are prevented from returning a 451 response status.

RFC 2119 defines "MAY" as a keyword, which may cause confusion with "may
not", even though the latter is not uppercased.

This hopefully better clarifies that the server in question may or may
not be an origin server, as opposed to implying that origin servers are
prevented from returning a 451 response status.
@aprescott
Copy link
Author

I re-wrapped the content to 80 characters after making this change, although I'm happy to undo that if the diff noise is unwelcome.

I also would be glad to go with a different choice of wording if there's something more preferable.

@reschke reschke added the 451 label May 1, 2015
@reschke
Copy link
Contributor

reschke commented Aug 14, 2015

Sounds good to me.

@mnot mnot closed this Dec 30, 2015
@aprescott
Copy link
Author

Out of curiosity, why this was closed?

@mnot
Copy link
Member

mnot commented Jan 4, 2016

because the RFC has been published.

@reschke
Copy link
Contributor

reschke commented Jan 4, 2016

Actually, it has been approved but not published, and this is a change that could be done during AUTH48 (a minimal change would be "may" -> "might").

@mnot
Copy link
Member

mnot commented Jan 4, 2016

@timbray, thoughts? I think this is editors' discretion (although historically we do try to avoid using RFC2119 keywords in this manner).

@aprescott aprescott deleted the may-not-ambiguity branch January 7, 2016 15:36
@aprescott
Copy link
Author

Happy to resubmit a PR with the change, unless @timbray or someone else would just make that small modification.

@reschke
Copy link
Contributor

reschke commented Feb 17, 2016

I think this can be considered to be resolved due to the changes in f36fb4a.

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

Successfully merging this pull request may close these issues.

None yet

3 participants