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: Catalyst@lists.rawmode.org Listinfo: http://lists.rawmode.org/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/catalyst@lists.rawmode.org/ Dev site: http://dev.catalyst.perl.org/
