[java-identity-provider] 11/16: Include JAAS config name in success/failure logging.

[Scott Cantor]

This is an automated email from the git hooks/post-receive script.

scantor pushed a commit to branch maint-3.3
in repository java-identity-provider.

View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=d03c1a255480df4f681e87c02798cd92af05b417

commit d03c1a255480df4f681e87c02798cd92af05b417
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Wed May 31 15:12:32 2017 -0400

    Include JAAS config name in success/failure logging.
---
 .../idp/authn/impl/ValidateUsernamePasswordAgainstJAAS.java    | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ValidateUsernamePasswordAgainstJAAS.java b/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ValidateUsernamePasswordAgainstJAAS.java
index b07e2fb..a189933 100644
--- a/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ValidateUsernamePasswordAgainstJAAS.java
+++ b/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ValidateUsernamePasswordAgainstJAAS.java
@@ -243,20 +243,22 @@ public class ValidateUsernamePasswordAgainstJAAS extends AbstractUsernamePasswor
                 log.debug("{} Attempting to authenticate user '{}' via '{}'", getLogPrefix(),
                         getUsernamePasswordContext().getUsername(), currentLoginConfigName);
                 authenticate(currentLoginConfigName);
-                log.info("{} Login by '{}' succeeded", getLogPrefix(), getUsernamePasswordContext().getUsername());
+                log.info("{} Login by '{}' via '{}' succeeded", getLogPrefix(),
+                        getUsernamePasswordContext().getUsername(), currentLoginConfigName);
                 recordSuccess(profileRequestContext);
                 derivedSubject = loginConfig.getSecond();
                 buildAuthenticationResult(profileRequestContext, authenticationContext);
                 ActionSupport.buildProceedEvent(profileRequestContext);
                 return;
             } catch (final LoginException e){ 
-                log.info("{} Login by '{}' failed", getLogPrefix(), getUsernamePasswordContext().getUsername(), e);
+                log.info("{} Login by '{}' via '{}' failed", getLogPrefix(), getUsernamePasswordContext().getUsername(),
+                        currentLoginConfigName, e);
                 handleError(profileRequestContext, authenticationContext, e, AuthnEventIds.INVALID_CREDENTIALS);
                 recordFailure(profileRequestContext, true);
                 eventSignaled = true;
             } catch (final Exception e) {
-                log.warn("{} Login by '{}' produced exception", getLogPrefix(),
-                        getUsernamePasswordContext().getUsername(), e);
+                log.warn("{} Login by '{}' via '{}' produced exception", getLogPrefix(),
+                        getUsernamePasswordContext().getUsername(), currentLoginConfigName, e);
                 handleError(profileRequestContext, authenticationContext, e, AuthnEventIds.AUTHN_EXCEPTION);
                 recordFailure(profileRequestContext, false);
                 eventSignaled = true;

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.