You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2011/10/22 19:08:18 UTC

svn commit: r1187732 - /myfaces/core/branches/2.0.x/shared/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java

Author: lu4242
Date: Sat Oct 22 17:08:17 2011
New Revision: 1187732

URL: http://svn.apache.org/viewvc?rev=1187732&view=rev
Log:
MYFACES-3354 NullPointerException on jetty 6.1.5 with faces-redirect=true action result

Modified:
    myfaces/core/branches/2.0.x/shared/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java

Modified: myfaces/core/branches/2.0.x/shared/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/2.0.x/shared/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java?rev=1187732&r1=1187731&r2=1187732&view=diff
==============================================================================
--- myfaces/core/branches/2.0.x/shared/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java (original)
+++ myfaces/core/branches/2.0.x/shared/src/main/java/org/apache/myfaces/shared/context/flash/FlashImpl.java Sat Oct 22 17:08:17 2011
@@ -631,7 +631,9 @@ public class FlashImpl extends Flash
                 // to this request object
                 cookie.setMaxAge(0);
                 cookie.setPath(_getCookiePath(externalContext));
-                cookie.setValue(null);
+                //MYFACES-3354 jetty 6.1.5 does not allow this,
+                //call setMaxAge(0) is enough
+                //cookie.setValue(null);
                 httpResponse.addCookie(cookie);
             }
         }