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

GZIP in DATA #463

Merged
merged 3 commits into from
Apr 23, 2014
Merged

GZIP in DATA #463

merged 3 commits into from
Apr 23, 2014

Conversation

phluid61
Copy link
Contributor

A proposal for adding GZIP compression to DATA frames. One possible resolution for #445.

@martinthomson
Copy link
Collaborator

This looks good. Thanks for putting this together. We'll see how the discussion progresses.

@phluid61
Copy link
Contributor Author

I realise we might also want to modify Section 8.1.3.5 Malformed Messages, to explain that Content-Length either cannot be sent if any frame is compressed, or measures the length after inflation, depending on how closely this maps to transfer encoding (http://tools.ietf.org/html/draft-ietf-httpbis-p1-messaging-26#section-3.3.2).

I suppose that's a follow-up issue for the WG to discuss.

@martinthomson
Copy link
Collaborator

No. You are right. Content-length applies pre-compression or
post-decompression. We need to that clear.
On Apr 22, 2014 4:28 PM, "Matthew Kerwin" [email protected] wrote:

I realise we might also want to modify Section 8.1.3.5 Malformed
Messages
, to explain that Content-Length either cannot be sent if any
frame is compressed, or measures the length after inflation, depending on
how closely this maps to transfer encoding (
http://tools.ietf.org/html/draft-ietf-httpbis-p1-messaging-26#section-3.3.2
).

I suppose that's a follow-up issue for the WG to discuss.


Reply to this email directly or view it on GitHubhttps://github.com//pull/463#issuecomment-41108445
.

@martinthomson martinthomson merged commit 1580ac1 into httpwg:master Apr 23, 2014
@phluid61 phluid61 deleted the 445-gzip branch April 23, 2014 23:56
@cben cben mentioned this pull request 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 this pull request may close these issues.

2 participants