Follow HTTP 308 redirects

benoitc / hackney

simple HTTP client in Erlang

Other

1.34k stars 427 forks source link

Follow HTTP 308 redirects #692

Closed AntoineAugusti closed 1 year ago

AntoineAugusti commented 3 years ago

Fixes #635

I'm not sure if I need to make other changes. Did a few searches on the codebase and did not find obvious things to change. Let me know if I missed something.

thbar commented 2 years ago

Hello @benoitc! We would love to get your opinion about this PR, and know how to help out here. Let us know if we can do something!

AntoineAugusti commented 2 years ago

@benoitc Wondering if we can do this regarding this PR!

AntoineAugusti commented 1 year ago

Hi @benoitc, we would love to get this change merged as we require it upstream ❤️

fchabouis commented 1 year ago

Hi, I would be glad too if this was merged! Thanks :)

benoitc commented 1 year ago

there is a coming release tomorrow. This will be part of it. Thanks for the patch!