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 GZIP and Deflate #404

Closed
mnot opened this issue Feb 20, 2014 · 5 comments
Closed

Requiring GZIP and Deflate #404

mnot opened this issue Feb 20, 2014 · 5 comments

Comments

@mnot
Copy link
Member

mnot commented Feb 20, 2014

9.3 GZip Content-Encoding says:

Clients MUST support gzip compression for HTTP request bodies. Regardless of the value of the accept-encoding header field, a server MAY send responses with gzip or deflate encoding.

A few things here:

  1. My understanding is that people want this so that intermediaries can't strip encoding and therefore impact performance. However, AIUI those same people intend to always run HTTP/2 over TLS, so I wonder if this is really necessary.

If we keep this text, a few requests:

  1. Is it both gzip and deflate (which last I checked, some clients don't support)? If so, the first sentence and section title should be changed to reflect this.
  2. The implication on a HTTP/1 to HTTP/2 intermediary should be spelled out -- that they MUST decompress the response body if the HTTP/1 client doesn't advertise support for the compression used.
@martinthomson
Copy link
Collaborator

Sounds reasonable. I assume the design issue here for discussion is the first (i.e., gzip or gzip+deflate) ?

@jpinner
Copy link

jpinner commented Feb 20, 2014

My understanding of the intent is that both gzip and deflate encoding must be supported by the client -- but happy to ask the list :)

@grmocg
Copy link
Contributor

grmocg commented Feb 20, 2014

That was the original intent.
I can't think of a truly good reason that both MUST be supported (so long
as one is), but most libraries do both if they do one..

On Wed, Feb 19, 2014 at 9:40 PM, Jeff Pinner [email protected]:

My understanding of the intent is that both gzip and deflate encoding must
be supported by the client -- but happy to ask the list :)

Reply to this email directly or view it on GitHubhttps://github.com//issues/404#issuecomment-35590944
.

@mnot
Copy link
Member Author

mnot commented Feb 20, 2014

@martinthomson - yes. Will probably just bring it up in London, given the timing.

@mnot
Copy link
Member Author

mnot commented Mar 3, 2014

Discussed in London; gzip only + editorial massaging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants