There are two issues open for going over this part. I hope to have them resolved this week to cover all cases. It makes sense to anchor them, but I believe we found an issue which made us change it. Should dig through the Git history when doing this. On Mon, Dec 15, 2014 at 10:02 AM, Gabor Szabo <gabor@szabgab.com> wrote:
I just found an issue with my D1 => D2 migration that broke the Perl Maven site. I think I managed to fix it but I'd like to clarify this:
I used to have a route handler like this:
get qr{/pro/(.*)} => sub {
my ($path) = splat;
return $path;
}; In D1 this was matching only routes that started with /pro As if there was an implicit ^ at the beginning of the route.
Now, in D2, it will match anywhere so it started to match URLs such as /media/pro/file
I could fix this by adding an explicit caret to the beginning of the regex:
get qr{^/pro/(.*)} => sub {
my ($path) = splat;
return $path;
};
Please confirm if this is really the case or if I just had some other issues? I think this should be added to the migration document with the recommended solution.
Could I have solved this with the "prefix" keyword? How? Would that be better?
regards Gabor
-- Gabor Szabo http://szabgab.com/ Perl Maven http://perlmaven.com/ Perl Weekly http://perlweekly.com/ Phone: +972-54-4624648
_______________________________________________ dancer-users mailing list dancer-users@dancer.pm http://lists.preshweb.co.uk/mailman/listinfo/dancer-users