-
Notifications
You must be signed in to change notification settings - Fork 23.1k
docs(Fx132): HTTP/2 Server Push is now deactivated by default in most major browsers #36149
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
Conversation
|
This pull request has merge conflicts that must be resolved before it can be merged. |
| For now, there are a few things to keep in mind: | ||
|
|
||
| - Usually only _clients_ make HTTP requests, and only to _servers_. Servers _respond_ to a _client_'s HTTP request. A server can also populate data into a client cache, in advance of it being requested, through a mechanism called [server push](https://en.wikipedia.org/wiki/HTTP/2_Server_Push). | ||
| - _Clients_ make HTTP requests to _servers_. Servers _respond_ to a _client_'s HTTP request. |
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.
Losing the information about server push good. I don't think there are other HTTP mechanisms for a server to push information to a client without their first being a request.
Though FYI it is possible for a server to include information in a response that was not requested which "somewhat" does the same thing (i.e. server provides a hint to client in any response that is unrelated to the request, triggering behaviour that looks a bit like a push). Upshot not worth mentioning here.
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.
Right, I think for the target audience of this doc, we can keep it straightforward.
hamishwillee
left a comment
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.
A couple of "to my taste" changes. Approving so you can merge or accept if you like.
Note that I only reviewed the changes that were made - I haven't checked if there are other mentions of server push that should be updated.
Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
|
Thanks, Hamish!
I can't find any through code search, so I believe this is everything. I'll merge shortly 👍🏻 |
HTTP/2 Server Push is being unshipped everywhere. I'm checking everywhere we mention "Server Push" to make sure we're not recommending it anymore.
https://groups.google.com/a/mozilla.org/g/dev-platform/c/vU9hJg343U8/m/4cZsHz7TAQAJ
Related issues and pull requests: