From 7c5592f047b681be489e8abe06000df7f1af2ba9 Mon Sep 17 00:00:00 2001 From: jensp Date: Thu, 29 Aug 2019 08:52:02 +0000 Subject: [PATCH] SAML support for CCM git-svn-id: https://svn.libreccm.org/ccm/trunk@6168 8810af33-2d31-482b-a856-94f89814c4df --- .../src/com/arsdigita/kernel/security/SamlLoginModule.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/ccm-core/src/com/arsdigita/kernel/security/SamlLoginModule.java b/ccm-core/src/com/arsdigita/kernel/security/SamlLoginModule.java index 24c5f380d..df14a3fdc 100644 --- a/ccm-core/src/com/arsdigita/kernel/security/SamlLoginModule.java +++ b/ccm-core/src/com/arsdigita/kernel/security/SamlLoginModule.java @@ -63,7 +63,7 @@ public class SamlLoginModule implements LoginModule { auth = new Auth(OneLoginUtil.buildSettings(httpRequest), httpRequest, httpResponse); - } catch (SettingsException ex) { + } catch (IOException | SettingsException | Error ex) { LOGGER.error("SAML Login failed.", ex); throw new LoginException("SAML Login failed. Configuration error?"); } @@ -100,6 +100,10 @@ public class SamlLoginModule implements LoginModule { @Override public boolean logout() throws LoginException { + getRequest().getSession().invalidate(); + return true; + } + protected HttpServletRequest getRequest() throws LoginException { try { @@ -142,5 +146,4 @@ public class SamlLoginModule implements LoginModule { ); } } - }