Merge pull request #188 from eghm/master

Log an error message if CAS principal has no corresponding Confluence principal
This commit is contained in:
Misagh Moayyed 2017-06-20 09:18:46 -07:00 committed by GitHub
commit 9d57eb3655
1 changed files with 5 additions and 0 deletions

View File

@ -59,6 +59,11 @@ public final class ConfluenceCasAuthenticator extends ConfluenceAuthenticator {
if (assertion != null) {
final Principal p = getUser(assertion.getPrincipal().getName());
// user doesn't exist
if (p == null) {
LOGGER.error("Principal is null for ", assertion.getPrincipal().getName());
}
LOGGER.debug("Logging in [{}] from CAS.", p.getName());
session.setAttribute(LOGGED_IN_KEY, p);