[java-identity-provider COMMIT] /trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/UpdateIdPSessionWi...

noreply at shibboleth.net noreply at shibboleth.net
Tue Oct 18 16:24:56 EDT 2016


Author: scantor
Date: Tue Oct 18 16:24:56 2016
New Revision: 8507

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=8507&view=rev
Log:
IDP-996 - CAS with HTML local storage ServiceNotSpecified error

https://issues.shibboleth.net/jira/browse/IDP-996

Revert to catching only ResolverException.

Modified:
    trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/UpdateIdPSessionWithSPSessionAction.java

Modified: trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/UpdateIdPSessionWithSPSessionAction.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/UpdateIdPSessionWithSPSessionAction.java?rev=8507&r1=8506&r2=8507&view=diff
==============================================================================
--- trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/UpdateIdPSessionWithSPSessionAction.java	(original)
+++ trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/UpdateIdPSessionWithSPSessionAction.java	Tue Oct 18 16:24:56 2016
@@ -31,6 +31,8 @@
 import net.shibboleth.utilities.java.support.annotation.constraint.Positive;
 import net.shibboleth.utilities.java.support.logic.Constraint;
 import net.shibboleth.utilities.java.support.resolver.CriteriaSet;
+import net.shibboleth.utilities.java.support.resolver.ResolverException;
+
 import org.opensaml.profile.context.ProfileRequestContext;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -89,8 +91,8 @@
         try {
             log.debug("Attempting to retrieve session {}", ticket.getSessionId());
             session = sessionResolver.resolveSingle(new CriteriaSet(new SessionIdCriterion(ticket.getSessionId())));
-        } catch (final Exception e) {
-            log.warn("IdPSession resolution error: {}. Possible sign of misconfiguration.", e.getMessage());
+        } catch (final ResolverException e) {
+            log.warn("Possible sign of misconfiguration, IdPSession resolution error: {}", e);
         }
         if (session != null) {
             final long now = System.currentTimeMillis();



More information about the commits mailing list