Add support for Brotli compression #650
Conversation
|
This is great! I made a few changes to ensure Brotli is an optional dependency, otherwise I'm happy with this for initial (experimental?) support. Do you happen to know of a server that is problematic in this way, so I can manually test against a real-world scenario? |
- move dev deps to the Gemfile - brotli is not a dev dep for jruby (which it doesn't support) - test how brotli responses are handled when Brotli is not loaded
|
I am not able to give you the original URL due to non-disclosure concerns, but I found a random one that exhibits the same behavor. Here's the snippet that causes crash in Mechanize 2.11.0: P.S. Told you you will butcher my code to the point of uncrecognizeability! =^.~= |
I really wish you wouldn't do this. Your implementation and the initial test was fine and saved me a lot of time. Thank you. Thanks for the link, I've manually verified this seems to work and will ship a release today. |
2 participants
Closes #649