-
Notifications
You must be signed in to change notification settings - Fork 129
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
Expose acceptsGzip #34
Comments
I wouldn't be against making acceptsGzip/parseEncodings/parseCoding public, but we should move them to a subpackage if we do so. For the latter two, we might want to consider dropping our impl in favor of gddo/httputil/header.ParseAccept anyway, which would make the former trivial. |
In general it's hard to use the exported GzipResponseWriter directly because of this issue but also because there's no way to set its options from outside the package. |
I'd like to use this package with my custom web application wrapper, but I can't match the functionality provided by the
NewGzipLevelHandler
without theacceptsGzip
function exposed: https://github.com/NYTimes/gziphandler/blob/fb3533722e14198abe471546c9798fd556531451/gzip.go#L174The text was updated successfully, but these errors were encountered: