[Catalyst-commits] r12089 - Catalyst-Runtime/5.80/branches/fix_path_info_decoding

t0m at dev.catalyst.perl.org t0m at dev.catalyst.perl.org
Mon Nov 30 23:23:39 GMT 2009


Author: t0m
Date: 2009-11-30 23:23:38 +0000 (Mon, 30 Nov 2009)
New Revision: 12089

Modified:
   Catalyst-Runtime/5.80/branches/fix_path_info_decoding/Changes
Log:
Rewrite changelog entry to be more complete

Modified: Catalyst-Runtime/5.80/branches/fix_path_info_decoding/Changes
===================================================================
--- Catalyst-Runtime/5.80/branches/fix_path_info_decoding/Changes	2009-11-30 23:20:12 UTC (rev 12088)
+++ Catalyst-Runtime/5.80/branches/fix_path_info_decoding/Changes	2009-11-30 23:23:38 UTC (rev 12089)
@@ -10,9 +10,12 @@
    - Set $ENV{PATH_INFO} from $ENV{REQUEST_URI} combined with
      $ENV{SCRIPT_NAME} if possible. This is many web servers always fully
      decode PATH_INFO including URI reserved characters. This allows us to
-     tell foo%2cbar from foo%252cbar.. RT#50082
+     tell foo%2cbar from foo%252cbar, and fixes issues with %2F in paths
+     being incorrectly decoded, resulting in too many path parts (rather
+     than 1 path part containing a /, on some web servers (at least nginx).
+     (RT#50082)
    - Require new HTTP::Request::AsCGI so that it fully decodes $ENV{PATH_INFO}
-     in non CGI contexts.
+     in non CGI contexts. (RT#50082)
 
   Refactoring / cleanups:
    - NoTabs and Pod tests moved to t/author so that they're not run




More information about the Catalyst-commits mailing list