You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by mm...@apache.org on 2006/09/19 07:16:45 UTC

svn commit: r447739 - /myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/webapp/filter/JavaScriptDetectorFilter.java

Author: mmarinschek
Date: Mon Sep 18 22:16:45 2006
New Revision: 447739

URL: http://svn.apache.org/viewvc?view=rev&rev=447739
Log:
fix for [TOMAHAWK-531] JavascriptDetectorFilter: endless redirects with cookies disabled. Thanks to Markus Bussemer.

Modified:
    myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/webapp/filter/JavaScriptDetectorFilter.java

Modified: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/webapp/filter/JavaScriptDetectorFilter.java
URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/webapp/filter/JavaScriptDetectorFilter.java?view=diff&rev=447739&r1=447738&r2=447739
==============================================================================
--- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/webapp/filter/JavaScriptDetectorFilter.java (original)
+++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/webapp/filter/JavaScriptDetectorFilter.java Mon Sep 18 22:16:45 2006
@@ -53,8 +53,9 @@
 
         JavascriptUtils.setJavascriptDetected(request.getSession(true), true); // mark the session to use javascript
 
-        log.info("Enabled JavaScript for session - redirect to" + request.getParameter("goto"));
-        response.sendRedirect(request.getParameter("goto"));
+        String redirectURL = response.encodeRedirectURL(request.getParameter("goto"));
+        log.info("Enabled JavaScript for session - redirect to " + redirectURL);
+        response.sendRedirect(redirectURL);
     }
 
     public void destroy()