[Jifty-commit] r5542 - in jifty/trunk: lib/Jifty

[Jifty commits]

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(