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 not supported #2

Closed
greim opened this issue Dec 3, 2010 · 3 comments
Closed

gzip not supported #2

greim opened this issue Dec 3, 2010 · 3 comments

Comments

@greim
Copy link
Owner

greim commented Dec 3, 2010

Hoxy strips out the accept-encoding header so theoretically this shouldn't be an issue. However google appears to gzip responses anyway. This is only an issue when you try to read/write the response body, but then it's a very bad issue.

@greim
Copy link
Owner Author

greim commented Dec 13, 2010

Somewhere along the way the google thing has ceased being an issue. As long as lack of gzip accept encoding continues not being an issue, this bug won't be a priority.

[edit] The google thing seems intermittent as far as I can tell.

@greim
Copy link
Owner Author

greim commented Dec 22, 2010

There's basically one way to gzip with node and that's node-compress. My currently favored/contemplated solution is to check for the existence of that lib and use it if available, printing out a warning on startup if it isn't.

@greim
Copy link
Owner Author

greim commented Feb 13, 2014

Gzip support is in 1.0.

@greim greim closed this as completed Feb 13, 2014
greim pushed a commit that referenced this issue Jul 23, 2015
Merge latest changes from greim/hoxy
theikkila referenced this issue in theikkila/hoxy Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant