The current implementation doesn't really make brotli optional and the middleware will fail if it is missing and the server replies with brotli encoding.
This PR should make it optional but with the current specs it is a bit difficult to test it.
optional_dependency could be moved to Faraday::Middleware if it makes sense later.
Issue: #185
The current implementation doesn't really make
brotlioptional and the middleware will fail if it is missing and the server replies with brotli encoding.This PR should make it optional but with the current specs it is a bit difficult to test it.
optional_dependencycould be moved toFaraday::Middlewareif it makes sense later.This is just a possible solution. :)