Dave Rolsky wrote:
> On Fri, 27 Apr 2007, Bill Moseley wrote:
>
>> sub post_redirect {
>> my ( $c, $location ) = @_;
>> my ($version) = $c->request->protocol =~ m/(\d+\.\d+)/;
>>
>> # Make location absolute
>> $location = $c->uri_for( $location )
>> unless $location =~ /http/;
>
> I'd make that regex /^http/
>
> It's not inconceivable there could be a path with http in it.Well, if it's going to be THAT kind of party... ^ should really be \A :-)
signature.asc
Description: OpenPGP digital signature
_______________________________________________ List: [email protected] Listinfo: http://lists.rawmode.org/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/[email protected]/ Dev site: http://dev.catalyst.perl.org/
