You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2009/09/18 13:33:46 UTC

svn commit: r816585 - /sling/trunk/bundles/servlets/get/src/main/java/org/apache/sling/servlets/get/impl/RedirectServlet.java

Author: cziegeler
Date: Fri Sep 18 11:33:45 2009
New Revision: 816585

URL: http://svn.apache.org/viewvc?rev=816585&view=rev
Log:
SLING-1115 : Redirecting to an absolute url does not work

Modified:
    sling/trunk/bundles/servlets/get/src/main/java/org/apache/sling/servlets/get/impl/RedirectServlet.java

Modified: sling/trunk/bundles/servlets/get/src/main/java/org/apache/sling/servlets/get/impl/RedirectServlet.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/get/src/main/java/org/apache/sling/servlets/get/impl/RedirectServlet.java?rev=816585&r1=816584&r2=816585&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/get/src/main/java/org/apache/sling/servlets/get/impl/RedirectServlet.java (original)
+++ sling/trunk/bundles/servlets/get/src/main/java/org/apache/sling/servlets/get/impl/RedirectServlet.java Fri Sep 18 11:33:45 2009
@@ -123,6 +123,12 @@
      */
     protected static String toRedirectPath(String targetPath,
             SlingHttpServletRequest request) {
+        // first check for an absolute path
+        final int protocolIndex = targetPath.indexOf(":/");
+        final int queryIndex = targetPath.indexOf('?');
+        if (  protocolIndex > -1 && (queryIndex == -1 || queryIndex > protocolIndex) ) {
+            return targetPath;
+        }
 
         String postFix;
         RequestPathInfo rpi = request.getRequestPathInfo();