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

Requiring Clients to support content-codings #460

Closed
mnot opened this issue Apr 18, 2014 · 4 comments · Fixed by #495
Closed

Requiring Clients to support content-codings #460

mnot opened this issue Apr 18, 2014 · 4 comments · Fixed by #495

Comments

@mnot
Copy link
Member

mnot commented Apr 18, 2014

HTTP/2 requires clients to support the gzip content-coding.

The justification for this has always been that it helps avoid intermediaries and other interposed software (e.g., virus filters) that strip Accept-Encoding to avoid the pain of decompressing responses.

However, it has been pointed out that doing so means that intermediaries that translate from 1 to 2 are now required to synthesise new entity tags for decompressed responses, breaking semantic transparency and/or losing significant HTTP functionality.

@mnot
Copy link
Member Author

mnot commented Apr 18, 2014

See also #404.

@reschke
Copy link
Contributor

reschke commented Apr 28, 2014

It also requires to rewrite ETags in request headers (conditional ones), which is an open-ended list. Furthermore, ETags also appear in payloads (WebDAV properties, for instance).

@phluid61
Copy link
Contributor

The outcome of this issue also affects #424

@mnot
Copy link
Member Author

mnot commented Jun 5, 2014

Discussed in NYC; will remove implicit gzip from spec; may create new content-coding required status code or other strategy (non-blocking).

martinthomson added a commit that referenced this issue Jun 5, 2014
Removing requirements on Content-Encoding.  Closes #460
@cben cben mentioned this issue Mar 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants