[Catalyst] Follow-up to new uri_for() bug

apv apv at sedition.com
Tue Sep 12 03:01:37 CEST 2006

009531.html (I did start a new message there, blame Mail.app, not me  
for the bad threading)

Okay, mean guys. Make me solve my own, er, Catalyst's own, bugs.

Line 118 (5.7001) of Catalyst::Engine::CGI looks like this:
         $base_path =~ s/$ENV{PATH_INFO}$//;

Unless I'm losing it, it should look like this:
         $base_path =~ s/\Q$ENV{PATH_INFO}\E$//;

Otherwise uri_for (well, request->base and anything that uses it)  
gets borked by any number of regex chars in the URI.


More information about the Catalyst mailing list