[Catalyst-commits] r9210 - in Catalyst-Runtime/5.80/trunk: . lib/Catalyst

[rafl at dev.catalyst.perl.org]

Author: rafl
Date: 2009-02-05 14:01:11 +0000 (Thu, 05 Feb 2009)
New Revision: 9210

Modified:
   Catalyst-Runtime/5.80/trunk/Changes
   Catalyst-Runtime/5.80/trunk/lib/Catalyst/Request.pm
Log:
Clarify that request arguments aren't unescaped automatically.

Courtesy of Simon Bertrang. Closes RT#41153.

Modified: Catalyst-Runtime/5.80/trunk/Changes
===================================================================
--- Catalyst-Runtime/5.80/trunk/Changes	2009-02-05 12:59:03 UTC (rev 9209)
+++ Catalyst-Runtime/5.80/trunk/Changes	2009-02-05 14:01:11 UTC (rev 9210)
@@ -1,5 +1,8 @@
 # This file documents the revision history for Perl extension Catalyst.
 
+        - Clarify that request arguments aren't unescaped automatically
+          (Simon Bertrang) (Closes RT#41153)
+
 5.8000_06 2009-02-04 21:00
         - Disallow writing to config after setup (rafl)
         - Disallow calling setup more than once (rafl)

Modified: Catalyst-Runtime/5.80/trunk/lib/Catalyst/Request.pm
===================================================================
--- Catalyst-Runtime/5.80/trunk/lib/Catalyst/Request.pm	2009-02-05 12:59:03 UTC (rev 9209)
+++ Catalyst-Runtime/5.80/trunk/lib/Catalyst/Request.pm	2009-02-05 14:01:11 UTC (rev 9210)
@@ -210,6 +210,9 @@
 and the URI for the request was C<http://.../foo/moose/bah>, the string C<bah>
 would be the first and only argument.
 
+Arguments just get passed through and B<don't> get unescaped automatically, so
+you should do that explicitly.
+
 =head2 $req->args
 
 Shortcut for arguments.