[java-idp-plugin-duo] branch main updated: Improve Javadoc

Codeberg noreply at shibboleth.net
Thu Jul 16 10:40:17 UTC 2026


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

codeberg pushed a commit to branch main
in repository java-idp-plugin-duo.

View the commit online:
https://codeberg.org/Shibboleth/java-idp-plugin-duo/commit/0a6e7aebd0818a6fe2dd9ef9e7756d10e59c458d

The following commit(s) were added to refs/heads/main by this push:
     new 0a6e7aeb Improve Javadoc
0a6e7aeb is described below

commit 0a6e7aebd0818a6fe2dd9ef9e7756d10e59c458d
Author: Phil Smart <philip.smart at jisc.ac.uk>
AuthorDate: Thu Jul 16 11:40:09 2026 +0100

    Improve Javadoc
---
 .../duo/impl/AuthnMethodReferenceToPrincipalMappingStrategy.java   | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/AuthnMethodReferenceToPrincipalMappingStrategy.java b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/AuthnMethodReferenceToPrincipalMappingStrategy.java
index 52f4a7a0..e94f8af1 100644
--- a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/AuthnMethodReferenceToPrincipalMappingStrategy.java
+++ b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/AuthnMethodReferenceToPrincipalMappingStrategy.java
@@ -51,7 +51,8 @@ import net.shibboleth.shared.primitive.LoggerFactory;
  *  {@link ValidateDuoTokenAuthenticationResult}. It enables one or more {@link Principal}s to be associated with 
  *  each AMR value present in the authentication response.</p>
  */
-public class AuthnMethodReferenceToPrincipalMappingStrategy implements Function<ProfileRequestContext,Collection<Principal>>{
+public class AuthnMethodReferenceToPrincipalMappingStrategy 
+    implements Function<ProfileRequestContext,Collection<Principal>>{
     
     /** The name of the AMR claim inside the Duo authentication token.*/
     @Nonnull @NotEmpty private static final String AMR_CLAIM_NAME = "amr";
@@ -100,7 +101,7 @@ public class AuthnMethodReferenceToPrincipalMappingStrategy implements Function<
             final JWT token = duoContext.getAuthToken();
             tokenClaims = token != null ? token.getJWTClaimsSet() : null;
         } catch (final ParseException e) {
-            log.warn("Unable to parse token claims from authentication token, no AMRs can be mapped",e);
+            log.warn("Unable to parse token claims from authentication token; AMR mapping could not be performed", e);
             return CollectionSupport.emptyList();
         }
         if (tokenClaims == null) {
@@ -111,7 +112,7 @@ public class AuthnMethodReferenceToPrincipalMappingStrategy implements Function<
         try {
             amrs = tokenClaims.getStringListClaim(AMR_CLAIM_NAME);
         } catch (final ParseException e) {
-            log.warn("Unable to parse AMR claims from authentication token, no AMRs can be mapped",e);
+            log.warn("Unable to parse AMR claims from authentication token; AMR mapping could not be performed", e);
             return CollectionSupport.emptyList();
         }
         log.trace("Attempting to map AMR claims '{}'", amrs);

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


More information about the commits mailing list