-
Notifications
You must be signed in to change notification settings - Fork 33
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
encoding order wrong #10
Comments
I'm not sure this is a bug since I couldn't find anywhere in the rfc that says that order of appearance of the encodings is relevant. Did you run into a situation where this is an issue? |
I think I'll probably change it anyways because it's a simple change. |
yeah I'm not sure if it matters via RFC either but seems logical from a user perspective |
no mention of it, just "If multiple content-codings are acceptable, then the acceptable content-coding with the highest non-zero qvalue is preferred." - so I guess it's left to interpretation |
I just fixed it on master |
Accept-Encoding: gzip, deflate
is giving me['deflate', 'gzip']
with:The text was updated successfully, but these errors were encountered: