From: Tomas Doran <bobtfish@bobtfish.net>
Date: Wed, 5 Oct 2011 00:52:55 +0000 (+0100)
Subject: Clarify use of uri_for further. RT#57011
X-Git-Tag: 5.90003~4
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=commitdiff_plain;h=824a5eb06f0e83845f000a33124482be89c2291f

Clarify use of uri_for further. RT#57011
---

diff --git a/Changes b/Changes
index a72c75f..21883b6 100644
--- 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
--- 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 {