Skip Menu |
 

This queue is for tickets about the POE-Component-Client-HTTP CPAN distribution.

Report information
The Basics
Id: 14630
Status: resolved
Priority: 0/
Queue: POE-Component-Client-HTTP

People
Owner: RCAPUTO [...] cpan.org
Requestors: YKAR [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



Subject: FollowRedirects interpreted incorrectly
Download (untitled) / with headers
text/plain 300b
If to set FollowRedirects to 0, developer suppose to disable redirects but actually 0 means allow 1 redirect (1 means allow 2 redirects and so on). My patch will fix this (if FollowRedirects set to 0 no redirects will be allowed (302 status will be returned), if set to 1 then one redirect allowed).
Index: Request.pm =================================================================== --- Request.pm (revision 217) +++ Request.pm (working copy) @@ -418,7 +418,7 @@ last if ($history > $max); } - if ($history > $max) { + if ($history >= $max) { #$self->[REQ_STATE] |= RS_DONE; DEBUG and warn "RED: Too much redirection"; }
Applied. Thank you, and apologies for the wait.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.