Clarify use of uri_for further. RT#57011

diff --git a/Changes b/Changes
index a72c75f..21883b6 100644 (file)
--- a/Changes
+++ b/Changes
@@ -25,8 +25,11 @@
 
    - Document Catalyst::Plugin::Authentication fails tests unless
      you use the latest version with Catalyst 5.9
+
    - Clarify that prepare is called as a class method
 
+   - Clarify use of uri_for further. RT#57011
+
 5.90002 - 2011-08-22 21:44:00
   Backward compatibility fixes:
 

diff --git a/lib/Catalyst/Response.pm b/lib/Catalyst/Response.pm
index 834b349..1e1e4bf 100644 (file)
--- a/lib/Catalyst/Response.pm
+++ b/lib/Catalyst/Response.pm
@@ -149,6 +149,12 @@ redirect destination, and then sets the response status.  You will
 want to C< return > or C<< $c->detach() >> to interrupt the normal
 processing flow if you want the redirect to occur straight away.
 
+B<Note:> do not give a relative URL as $url, i.e: one that is not fully
+qualified (= C<http://...>, etc.) or that starts with a slash
+(= C</path/here>). While it may work, it is not guaranteed to do the right
+thing and is not a standard behaviour. You may opt to use uri_for() or
+uri_for_action() instead.
+
 =cut
 
 sub redirect {