-
Notifications
You must be signed in to change notification settings - Fork 606
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
allow disabling gzip in http #3231
Comments
Update: Just realized, it appears ETag is not exactly rewritten for gzipped responses - instead, it is prefixed with a weak validation specifier W/. The actual identifier part of etag is left as is. Which feels like exactly correct behaviour, and makes the problem less severe / easy to work around. I think the issue still stands; ability to disable the compression would be nice. |
Response auto-compression is implemented in Deno CLI, not std. Also, the option to disable gzip doesn't seem to exist on purpose. (denoland/deno#13769 (comment)) |
related denoland/deno#13830 |
I think this issue should be closed, as this is a concern with the runtime, not the Standard Library. |
Yeah, this is Deno CLI issue, not a std issue. Please track this in denoland/deno#13830 Closing |
Is your feature request related to a problem? Please describe.
Apparently the std http server implementation automagically applies gzip compression for some responses. (I'm guessing for responses that look like highly compressible text files).
While this might be desireable behaviour in most cases, there are some problems:
Describe the solution you'd like
While I personally think http compression should be opt-in and disabled by default. I would be happy to have at least a config option to disable it.
The text was updated successfully, but these errors were encountered: