[Jifty-commit] r5542 - in jifty/trunk: lib/Jifty
Jifty commits
jifty-commit at lists.jifty.org
Mon Jul 7 11:35:31 EDT 2008
Author: trs
Date: Mon Jul 7 11:35:30 2008
New Revision: 5542
Modified:
jifty/trunk/ (props changed)
jifty/trunk/lib/Jifty/Web.pm
Log:
r36895 at zot: tom | 2008-07-07 11:33:47 -0400
When setting the continuation request's path in a redirect, we need to unescape it to be consistent with Apache, lighttpd, and HTTP::Server::Simple
Modified: jifty/trunk/lib/Jifty/Web.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Web.pm (original)
+++ jifty/trunk/lib/Jifty/Web.pm Mon Jul 7 11:35:30 2008
@@ -730,7 +730,11 @@
}
my %parameters = ($page->parameters);
$request->argument($_ => $parameters{$_}) for keys %parameters;
- $request->path($page->url);
+
+ # Apache, lighttpd, and HSS all do one pass of unescaping on
+ # PATH_INFO, which is what $request->path is normally set to.
+ # We should replicate that here.
+ $request->path( URI::Escape::uri_unescape( $page->url ) );
$request->continuation($self->request->continuation);
my $cont = Jifty::Continuation->new(
More information about the Jifty-commit
mailing list