-
Notifications
You must be signed in to change notification settings - Fork 205
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
Conform to HTTP Style Guide #4977
Conversation
0afcd48
to
b395030
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent, I shall borrow the referencing markdown into my draft 😀
b395030
to
81e2b79
Compare
Note that while the style guide says to use 6265bis for COOKIES, I'm continuing to point to RFC 6265 unless and until we decide to take another delaying dependency. (I note that http-core also uses RFC 6265.) |
That sounds very sensible to me, especially given the context in which they are spoken to in this document. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Took a quick look - seems fine to me
Fixes #4951 and some other things besides. This presupposes a resolution of httpwg/admin#41 to be "treat them the same as regular fields," which is ugly but logically consistent. Easy enough to adjust if that isn't the eventual decision, of course.