search

chansen / p5-http-tiny

Tiny HTTP Client
https://metacpan.org/dist/HTTP-Tiny
53 stars 52 forks source link

“redirects” is lost if an internal exception happens #97

Closed FGasper closed 8 years ago

FGasper commented 8 years ago

Where request() returns status 599, it needs to add:

@{$args->{_redirects}} ? ('redirects'=>delete $args->{_redirects}) : ()
xdg commented 8 years ago

Shipped as HTTP-Tiny-0.067-TRIAL. If you can replicate, please try that and confirm that it works.

FGasper commented 8 years ago

Confirmed: This resolves the issue. Thank you!

FGasper commented 8 years ago

@xdg

xdg commented 8 years ago

Excellent! Thanks for testing it out. I'm going to let 0.067-TRIAL smoke for a few days and then will ship the stable release.

FGasper commented 8 years ago

@xdg Has this had long enough to incubate, do you think? We’d like to bundle the new version, but our infrastructure presumes we’re grabbing a production release.

xdg commented 8 years ago

Yes. Thanks for the reminder. 0.068 shipped to CPAN just now.