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.

Sign up for GitHub

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

Jump to bottom

Semantics talking about Host header field #511

Closed
kazuho opened this issue Nov 6, 2020 · 2 comments · Fixed by #575
Closed

Semantics talking about Host header field #511

kazuho opened this issue Nov 6, 2020 · 2 comments · Fixed by #575
Assignees
@royfielding
Labels
semantics

Comments

@kazuho
Copy link

kazuho commented Nov 6, 2020 •
edited
Loading

Recently I noticed that the host header field is being discussed in section 6.2.1 of the semantics draft.

I believe that we should

  • move the definition of the host header to the HTTP/1.1 messaging draft, and
  • discuss about "authority" as a concept in the semantics draft,

as the "host" header is specific to HTTP/1.1 (except for handling of broken HTTP/2 clients)?
@mnot mnot self-assigned this Nov 11, 2020
@mnot mnot added the semantics label Nov 11, 2020
@mnot
Copy link
Member

mnot commented Nov 12, 2020

Hi Kazuho,

We discussed this on the editors' call. Host's semantics aren't specific to HTTP/1.1, even if its requirements are different in other protocol versions. That's why we moved it to Semantics; it's not version-specific (and really very few headers should be).

Closing; if you feel it needs more discussion please request re-opening and we'll do so.

@mnot mnot closed this as completed Nov 12, 2020
@kazuho kazuho mentioned this issue Nov 12, 2020
Closed
@kazuho
Copy link
Author

kazuho commented Nov 12, 2020

Thank you for the consideration. While I do understand the argument that the semantics of the header field is not version specific, the use is. I've opened a PR that tries to clarify that.

@mnot mnot reopened this Nov 25, 2020
@mnot mnot assigned royfielding and unassigned mnot Nov 25, 2020
@royfielding royfielding mentioned this issue Dec 8, 2020
Merged
reschke added a commit that referenced this issue Dec 8, 2020
@reschke
adjust change log (for #511)
2a82b6f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@royfielding royfielding

Labels
semantics
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add :authority pseudo-header to Host discussion and make section applicable to both httpwg/http-core
3 participants
@kazuho @mnot @royfielding