You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2011/03/15 23:04:22 UTC
svn commit: r1081960 -
/wicket/trunk/wicket-core/src/main/java/org/apache/wicket/protocol/http/WebSession.java
Author: mgrigorov
Date: Tue Mar 15 22:04:22 2011
New Revision: 1081960
URL: http://svn.apache.org/viewvc?rev=1081960&view=rev
Log:
WICKET-3537 Wicket example "Library" - sign out not working
Do not try to set the empty cookie used for authorization second time in the same request.
Modified:
wicket/trunk/wicket-core/src/main/java/org/apache/wicket/protocol/http/WebSession.java
Modified: wicket/trunk/wicket-core/src/main/java/org/apache/wicket/protocol/http/WebSession.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-core/src/main/java/org/apache/wicket/protocol/http/WebSession.java?rev=1081960&r1=1081959&r2=1081960&view=diff
==============================================================================
--- wicket/trunk/wicket-core/src/main/java/org/apache/wicket/protocol/http/WebSession.java (original)
+++ wicket/trunk/wicket-core/src/main/java/org/apache/wicket/protocol/http/WebSession.java Tue Mar 15 22:04:22 2011
@@ -163,9 +163,12 @@ public class WebSession extends Session
@Override
public void invalidate()
{
- getApplication().getSecuritySettings().getAuthenticationStrategy().remove();
+ if (isSessionInvalidated() == false)
+ {
+ getApplication().getSecuritySettings().getAuthenticationStrategy().remove();
- super.invalidate();
+ super.invalidate();
+ }
}
/**