Bug #18244 for HTTP-Proxy: requests_redirectable option deviates from default (expected) behaviour

Sun Mar 19 11:48:45 2006 stennie [...] cpan.org - Ticket created

Subject:

requests_redirectable option deviates from default (expected) behaviour

Hi, Is there a reason that requests_redirectable has been explicitly overridden from the default (expected) behaviour: $self->agent->requests_redirectable( [] ); From the LWP::UserAgent doc page: This reads or sets the object's list of request names that $ua->redirect_ok(...) will allow redirection for. By default, this is ['GET', 'HEAD'], as per RFC 2616. Attached is a patch against 0.17 which removes the offending line ;-). Cheers, Stephen @@ -207,7 +207,6 @@ $self->_init_agent if ( !defined $self->agent ); # specific agent config - $self->agent->requests_redirectable( [] ); $self->agent->agent(''); # for TRACE support $self->agent->protocols_allowed( [qw( http https ftp gopher )] );

Sun Mar 19 12:00:54 2006 stennie [...] cpan.org - Correspondence added

Show quoted text

> From the LWP::UserAgent doc page: > This reads or sets the object's list of request names that $ua-

> >redirect_ok(...) will allow

> redirection for. By default, this is ['GET', 'HEAD'], as per RFC 2616.

Note .. if you want to support proxying webdav for svn, you need to replace: $self->agent->requests_redirectable( [] ); With: push @{ $self->agent->requests_redirectable }, 'PROPFIND'; Cheers, Stephen

Sun Mar 19 12:00:55 2006 stennie [...] cpan.org - Status changed from 'new' to 'open'

Sun Mar 19 12:02:34 2006 book [...] cpan.org - Taken

Sun Mar 19 12:07:00 2006 book [...] cpan.org - Correspondence added

Le Dim. Mar. 19 11:48:45 2006, stennie a écrit : Show quoted text

> Hi, > > Is there a reason that requests_redirectable has been explicitly > overridden from the default > (expected) behaviour: > $self->agent->requests_redirectable( [] );

Yes there is. By default, the proxy is supposed to handle single requests as single requests. So, no redirection is followed by default, but rather sent back to the client who should react to the 3xx response itself. That's also the reason why it uses simple_request() rather than request().

Sun Mar 19 12:07:01 2006 book [...] cpan.org - Status changed from 'open' to 'rejected'