You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@syncope.apache.org by il...@apache.org on 2018/06/11 07:13:32 UTC

[3/4] syncope git commit: [SYNCOPE-1318] The actual fix: in case of uncontrolled error, do proper logout

[SYNCOPE-1318] The actual fix: in case of uncontrolled error, do proper logout


Project: http://git-wip-us.apache.org/repos/asf/syncope/repo
Commit: http://git-wip-us.apache.org/repos/asf/syncope/commit/df5c7fe4
Tree: http://git-wip-us.apache.org/repos/asf/syncope/tree/df5c7fe4
Diff: http://git-wip-us.apache.org/repos/asf/syncope/diff/df5c7fe4

Branch: refs/heads/2_0_X
Commit: df5c7fe4df54e40b33ab252370d578d9b99dfcb5
Parents: 36d06f8
Author: Francesco Chicchiriccò <il...@apache.org>
Authored: Mon Jun 11 09:13:06 2018 +0200
Committer: Francesco Chicchiriccò <il...@apache.org>
Committed: Mon Jun 11 09:13:06 2018 +0200

----------------------------------------------------------------------
 .../console/SyncopeConsoleRequestCycleListener.java     |  3 +--
 .../syncope/client/console/SyncopeConsoleSession.java   | 12 ------------
 2 files changed, 1 insertion(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/syncope/blob/df5c7fe4/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleRequestCycleListener.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleRequestCycleListener.java b/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleRequestCycleListener.java
index 91e55e6..50c5748 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleRequestCycleListener.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleRequestCycleListener.java
@@ -99,8 +99,7 @@ public class SyncopeConsoleRequestCycleListener extends AbstractRequestCycleList
 
         if (errorPage instanceof Login) {
             try {
-                SyncopeConsoleSession.get().cleanup();
-                SyncopeConsoleSession.get().invalidateNow();
+                SyncopeConsoleSession.get().invalidate();
             } catch (Throwable t) {
                 // ignore
                 LOG.debug("Unexpected error while forcing logout after error", t);

http://git-wip-us.apache.org/repos/asf/syncope/blob/df5c7fe4/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleSession.java
----------------------------------------------------------------------
diff --git a/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleSession.java b/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleSession.java
index 74b1db0..aa8a28c 100644
--- a/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleSession.java
+++ b/client/console/src/main/java/org/apache/syncope/client/console/SyncopeConsoleSession.java
@@ -199,18 +199,6 @@ public class SyncopeConsoleSession extends AuthenticatedWebSession {
         super.invalidate();
     }
 
-    @Override
-    public void invalidateNow() {
-        if (getJWT() != null) {
-            if (client != null) {
-                client.logout();
-            }
-            cleanup();
-        }
-        executor.shutdown();
-        super.invalidateNow();
-    }
-
     public UserTO getSelfTO() {
         return selfTO;
     }