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

Add note about 103 and HTTP/2 #9563

Merged
merged 4 commits into from
Aug 1, 2023

Conversation

tunetheweb
Copy link
Contributor

@tunetheweb tunetheweb commented Jul 27, 2023

Adds a small note clarifying 103 Early Hints recommendation to only use with HTTP/2 or later unless it is known the implementation supports this, as noted in the IETF RFC, and explains why examples are still in HTTP/1.1-style.

(See WHATWG Working Mode: Changes for more details.)


/acknowledgements.html ( diff )
/semantics.html ( diff )

Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I didn't know that this was "HTTP/1.1-style notation"; I thought it was just HTTP notation.

Do you want to add yourself to the acknowledgments section?

@tunetheweb
Copy link
Contributor Author

tunetheweb commented Jul 31, 2023

LGTM! I didn't know that this was "HTTP/1.1-style notation"; I thought it was just HTTP notation.

This is HTTP/1-style:

103 Early Hint
Link: </image.png>; rel=preload as=image

200 OK
Content-Type: text/html

HTTP/2 doesn't have status name after the status code, and uses :status pseudo header so would be more like:

:status: 103
Link: </image.png>; rel=preload as=image

:status: 200
Content-Type: text/html

But the HTTP/1.1-style is so ingrained that that just looks weird to us!

But it was enough to cause confusion internally here as to "why are all the examples in HTTP/1 when no one supports that yet?"

Do you want to add yourself to the acknowledgments section?

Done!

@domenic domenic added the clarification Standard could be clearer label Aug 1, 2023
@domenic domenic merged commit 27a0b5d into whatwg:main Aug 1, 2023
1 check passed
@tunetheweb tunetheweb deleted the add-note-about-103-http2 branch August 5, 2023 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clarification Standard could be clearer
Development

Successfully merging this pull request may close these issues.

2 participants