-
-
Notifications
You must be signed in to change notification settings - Fork 223
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
Switch default encoding to gzip for now #1260
Conversation
martin/martin/src/srv/tiles.rs Lines 200 to 204 in 7107db5
Seems this should be updated |
martin/martin/src/srv/tiles.rs Lines 281 to 297 in 7107db5
And maybe more tests should be added or updated here. |
Thx @sharkAndshark! Turns out it was never properly working -- it was always using the list of accepted encoding and ignored the order of the supported encodings... So we may have to rework that code. I re-worked the tests and removed the confusing default, but this is currently broken. Will try to get back to it soonish |
21a0164
to
92bfa82
Compare
GZip is significantly faster, so for now lets use that until brotli project gains some performance, and until we get proper after-compression caching.
GZip is significantly faster, so for now lets use that until brotli project gains some performance, and until we get proper after-compression caching.