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);
}
}