[java-identity-provider] branch main updated: OSJ-354: Suffix the PROTOCOL_MESSAGE category with .SAML

[Brent Putman]

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

putmanb 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=9821eec76c42e790bae63b8d6e1ec8f24ed4d407

The following commit(s) were added to refs/heads/main by this push:
     new 9821eec76 OSJ-354: Suffix the PROTOCOL_MESSAGE category with .SAML
9821eec76 is described below

commit 9821eec76c42e790bae63b8d6e1ec8f24ed4d407
Author: Brent Putman <putmanb at georgetown.edu>
AuthorDate: Thu Aug 3 20:29:40 2023 -0400

    OSJ-354: Suffix the PROTOCOL_MESSAGE category with .SAML
    
    Change API so that subclasses supply the logging sub-category,
    rather than the full category.
    
    Update decoders of admin endpoints to log to ".ADMIN".
---
 .../net/shibboleth/idp/admin/impl/DumpConfigRequestDecoder.java     | 6 ++++++
 .../shibboleth/idp/profile/impl/MetadataQueryRequestDecoder.java    | 6 ++++++
 .../net/shibboleth/idp/profile/impl/ResolverTestRequestDecoder.java | 6 ++++++
 .../saml/profile/impl/BaseIdPInitiatedSSORequestMessageDecoder.java | 2 +-
 4 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/idp-admin-impl/src/main/java/net/shibboleth/idp/admin/impl/DumpConfigRequestDecoder.java b/idp-admin-impl/src/main/java/net/shibboleth/idp/admin/impl/DumpConfigRequestDecoder.java
index e57cbc303..2fd807555 100644
--- a/idp-admin-impl/src/main/java/net/shibboleth/idp/admin/impl/DumpConfigRequestDecoder.java
+++ b/idp-admin-impl/src/main/java/net/shibboleth/idp/admin/impl/DumpConfigRequestDecoder.java
@@ -55,6 +55,12 @@ public class DumpConfigRequestDecoder extends AbstractHttpServletRequestMessageD
     /** Name of the query parameter carrying the requester: {@value} . */
     @Nonnull @NotEmpty public static final String REQUESTER_ID_PARAM = "requester";
 
+    /** Constructor. */
+    public DumpConfigRequestDecoder() {
+        super();
+        setProtocolMessageLoggerSubCategory("ADMIN");
+    }
+
     /** {@inheritDoc} */
     @Override
     protected void doDecode() throws MessageDecodingException {
diff --git a/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/MetadataQueryRequestDecoder.java b/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/MetadataQueryRequestDecoder.java
index 822112c12..045232007 100644
--- a/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/MetadataQueryRequestDecoder.java
+++ b/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/MetadataQueryRequestDecoder.java
@@ -54,6 +54,12 @@ public class MetadataQueryRequestDecoder extends AbstractHttpServletRequestMessa
     /** Name of the query parameter carrying the detectDuplicateEntityIDs: {@value} . */
     @Nonnull @NotEmpty public static final String DETECT_DUPLICATES_PARAM= "detectDuplicateEntityIDs";
     
+    /** Constructor. */
+    public MetadataQueryRequestDecoder() {
+        super();
+        setProtocolMessageLoggerSubCategory("ADMIN");
+    }
+
     /** {@inheritDoc} */
     @Override
     protected void doDecode() throws MessageDecodingException {
diff --git a/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/ResolverTestRequestDecoder.java b/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/ResolverTestRequestDecoder.java
index ffe3b949c..80b8acd26 100644
--- a/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/ResolverTestRequestDecoder.java
+++ b/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/ResolverTestRequestDecoder.java
@@ -52,6 +52,12 @@ public class ResolverTestRequestDecoder extends AbstractHttpServletRequestMessag
     /** Name of the query parameter for the SAML 2 protocol: {@value} . */
     @Nonnull @NotEmpty public static final String SAML2_PARAM = "saml2";
 
+    /** Constructor. */
+    public ResolverTestRequestDecoder() {
+        super();
+        setProtocolMessageLoggerSubCategory("ADMIN");
+    }
+
     /** {@inheritDoc} */
     @Override
     protected void doDecode() throws MessageDecodingException {
diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/profile/impl/BaseIdPInitiatedSSORequestMessageDecoder.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/profile/impl/BaseIdPInitiatedSSORequestMessageDecoder.java
index f060c0e9f..bfe3f3d35 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/profile/impl/BaseIdPInitiatedSSORequestMessageDecoder.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/profile/impl/BaseIdPInitiatedSSORequestMessageDecoder.java
@@ -61,7 +61,7 @@ public abstract class BaseIdPInitiatedSSORequestMessageDecoder extends AbstractH
     /** Constructor. */
     public BaseIdPInitiatedSSORequestMessageDecoder() {
         super();
-        setProtocolMessageLoggerCategory(BASE_PROTOCOL_MESSAGE_LOGGER_CATEGORY + ".SAML");
+        setProtocolMessageLoggerSubCategory("SAML");
     }
 
     /** ID generator. */

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