Skip to content
Snippets Groups Projects
Commit d4d8376a authored by brinn's avatar brinn
Browse files

Do not bail out on EnvironmentFailureException of...

Do not bail out on EnvironmentFailureException of authenticationService.check(), if the authentication service is remote, but just write a warning about the failed check into the log.

SVN: 28181
parent 18aa9a37
No related branches found
No related tags found
No related merge requests found
...@@ -174,7 +174,19 @@ public class DefaultSessionManager<T extends BasicSession> implements ISessionMa ...@@ -174,7 +174,19 @@ public class DefaultSessionManager<T extends BasicSession> implements ISessionMa
.getClass().getName())); .getClass().getName()));
operationLog.info(String.format("Session expiration period: %s", operationLog.info(String.format("Session expiration period: %s",
DurationFormatUtils.formatDurationHMS(sessionExpirationPeriodMillis))); DurationFormatUtils.formatDurationHMS(sessionExpirationPeriodMillis)));
authenticationService.check(); try
{
authenticationService.check();
} catch (EnvironmentFailureException ex)
{
if (authenticationService.isRemote())
{
operationLog.warn("Remote authentication service check failed.", ex);
} else
{
throw ex;
}
}
} }
private final T createAndStoreSession(final String user, final Principal principal, private final T createAndStoreSession(final String user, final Principal principal,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment