Include content_handlers in opts

ninenines / gun

HTTP/1.1, HTTP/2, Websocket client (and more) for Erlang/OTP.

ISC License

891 stars 232 forks source link

Include content_handlers in opts #242

Closed tomas-abrahamsson closed 3 years ago

tomas-abrahamsson commented 3 years ago

This fixes a dialyzer issue when using the gun and setting the content_handlers option.

tomas-abrahamsson commented 3 years ago

Oops, just force-pushed; fixed a gun_content_handler module name.

essen commented 3 years ago

Hey, I would like to document content handlers at the same time, but there's still an unknown that should be resolved first (trailers should probably be passed to content handlers).

essen commented 3 years ago

Heh going to merge it now anyway. Thanks!