[java-identity-provider] branch main updated: Add execution key to logging.

[Scott Cantor]

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

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

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

The following commit(s) were added to refs/heads/main by this push:
       new  35bd9e734 Add execution key to logging.
35bd9e734 is described below

commit 35bd9e73404ed8da91dd9333aec3f1228792f684
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Mon Aug 24 20:05:11 2020 -0400

    Add execution key to logging.
---
 .../idp/authn/impl/RemoteUserAuthServlet.java      | 24 +++++++++++-----------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/RemoteUserAuthServlet.java b/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/RemoteUserAuthServlet.java
index 54966f0a3..75da607df 100644
--- a/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/RemoteUserAuthServlet.java
+++ b/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/RemoteUserAuthServlet.java
@@ -239,12 +239,12 @@ public class RemoteUserAuthServlet extends HttpServlet {
             if (subjectAttribute != null) {
                 final Object subject = httpRequest.getAttribute(subjectAttribute);
                 if (subject != null && subject instanceof Subject) {
-                    log.debug("Java Subject extracted from attribute {}: {}", subjectAttribute, subject);
+                    log.debug("{}: Java Subject extracted from attribute {}: {}", key, subjectAttribute, subject);
                     httpRequest.setAttribute(ExternalAuthentication.SUBJECT_KEY, subject);
                     ExternalAuthentication.finishExternalAuthentication(key, httpRequest, httpResponse);
                     return;
                 }
-                log.info("Java Subject not found in attribute {}", subjectAttribute);
+                log.info("{}: Java Subject not found in attribute {}", key, subjectAttribute);
             }
             
             String username = null;
@@ -252,7 +252,7 @@ public class RemoteUserAuthServlet extends HttpServlet {
             if (checkRemoteUser) {
                 username = httpRequest.getRemoteUser();
                 if (username != null && !username.isEmpty()) {
-                    log.debug("User identity extracted from REMOTE_USER: {}", username);
+                    log.debug("{}: User identity extracted from REMOTE_USER: {}", key, username);
                 }
             }
             
@@ -261,7 +261,7 @@ public class RemoteUserAuthServlet extends HttpServlet {
                     final Object attr = httpRequest.getAttribute(s);
                     if (attr != null && !attr.toString().isEmpty()) {
                         username = attr.toString();
-                        log.debug("User identity extracted from attribute {}: {}", s, username);
+                        log.debug("{}: User identity extracted from attribute {}: {}", key, s, username);
                         break;
                     }
                 }
@@ -271,14 +271,14 @@ public class RemoteUserAuthServlet extends HttpServlet {
                 for (final String s : checkHeaders) {
                     username = httpRequest.getHeader(s);
                     if (username != null && !username.isEmpty()) {
-                        log.debug("User identity extracted from header {}: {}", s, username);
+                        log.debug("{}: User identity extracted from header {}: {}", key, s, username);
                         break;
                     }
                 }
             }
             
             if (username == null) {
-                log.info("User identity not found in request");
+                log.info("{}: User identity not found in request", key);
                 ExternalAuthentication.finishExternalAuthentication(key, httpRequest, httpResponse);
                 return;
             }
@@ -314,12 +314,12 @@ public class RemoteUserAuthServlet extends HttpServlet {
                             if (!Strings.isNullOrEmpty(method)) {
                                 final Principal p = getPrincipal(authnFlow, method);
                                 if (p != null) {
-                                    log.debug("Successfully processed authentication method from header {}: {}",
-                                            authnMethodHeader, method);
+                                    log.debug("{}: Successfully processed authentication method from header {}: {}",
+                                            key, authnMethodHeader, method);
                                     subject.getPrincipals().add(p);
                                 } else {
-                                    log.warn("Unable to locate a suitable Principal for authentication method " +
-                                            "from header {}: {}", authnMethodHeader, method);
+                                    log.warn("{}: Unable to locate a suitable Principal for authentication method " +
+                                            "from header {}: {}", key, authnMethodHeader, method);
                                 }
                             }
                         }
@@ -328,8 +328,8 @@ public class RemoteUserAuthServlet extends HttpServlet {
                         ExternalAuthentication.finishExternalAuthentication(key, httpRequest, httpResponse);
                         return;
                     }
-                    log.error("Unable to locate AuthenticationFlowDescriptor, can't process "
-                            + "authentication methods from header");
+                    log.error("{}: Unable to locate AuthenticationFlowDescriptor, can't process "
+                            + "authentication methods from header", key);
                 }
                 
                 httpRequest.setAttribute(ExternalAuthentication.PRINCIPAL_NAME_KEY, username);

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