[java-idp-plugin-duo] 05/16: Rename security context. Minor JavaDoc

Phil Smart philip.smart at jisc.ac.uk
Fri Oct 2 10:40:56 UTC 2020


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

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

View the commit online:
http://git.shibboleth.net/view/?p=java-idp-plugin-duo.git;a=commit;h=7b9d267f266227e3f2a04a85b609b7258b54ad0a

commit 7b9d267f266227e3f2a04a85b609b7258b54ad0a
Author: Phil Smart <philip.smart at jisc.ac.uk>
AuthorDate: Fri Sep 18 17:45:08 2020 +0100

    Rename security context. Minor JavaDoc
---
 .../shibboleth/idp/plugin/authn/duo/impl/DuoJWTClaimsVerifier.java  | 6 +++---
 ...WTSecurityContext.java => ProfileRequestJWTSecurityContext.java} | 4 ++--
 .../shibboleth/idp/plugin/authn/duo/impl/ValidateTokenClaims.java   | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/DuoJWTClaimsVerifier.java b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/DuoJWTClaimsVerifier.java
index 649dea8..52f987b 100644
--- a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/DuoJWTClaimsVerifier.java
+++ b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/DuoJWTClaimsVerifier.java
@@ -47,7 +47,7 @@ import net.shibboleth.utilities.java.support.logic.Constraint;
  *  </ol>
  */
 @ThreadSafe
-public class DuoJWTClaimsVerifier extends DefaultJWTClaimsVerifier<IdPJWTSecurityContext>{
+public class DuoJWTClaimsVerifier extends DefaultJWTClaimsVerifier<ProfileRequestJWTSecurityContext>{
     
     /** The name of the authentication time claim.*/
     @Nonnull public static final String AUTH_TIME_CLAIM_NAME = "auth_time";
@@ -105,7 +105,7 @@ public class DuoJWTClaimsVerifier extends DefaultJWTClaimsVerifier<IdPJWTSecurit
     }
     
     @Override
-    public void verify(@Nonnull final JWTClaimsSet claimsSet, @Nonnull final IdPJWTSecurityContext context)
+    public void verify(@Nonnull final JWTClaimsSet claimsSet, @Nonnull final ProfileRequestJWTSecurityContext context)
         throws BadJWTException {
         if (context == null) {
             throw new BadJWTException("Duo claims verifier requires the IdP security context");
@@ -116,7 +116,7 @@ public class DuoJWTClaimsVerifier extends DefaultJWTClaimsVerifier<IdPJWTSecurit
     }
     
     private void verifyAuthenticationTime(@Nonnull final JWTClaimsSet claimsSet, 
-            @Nonnull final IdPJWTSecurityContext context) throws BadJWTException {
+            @Nonnull final ProfileRequestJWTSecurityContext context) throws BadJWTException {
         
         if (context.getPrc() == null) {
             throw new BadJWTException("No profile request context found, can not validate authentication time");
diff --git a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/IdPJWTSecurityContext.java b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/ProfileRequestJWTSecurityContext.java
similarity index 91%
rename from idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/IdPJWTSecurityContext.java
rename to idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/ProfileRequestJWTSecurityContext.java
index 3fc594a..cfa8365 100644
--- a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/IdPJWTSecurityContext.java
+++ b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/ProfileRequestJWTSecurityContext.java
@@ -28,7 +28,7 @@ import net.shibboleth.utilities.java.support.logic.Constraint;
 
 /** Simple IdP Nimbus JWT Verification security context that holds the {@link ProfileRequestContext}.*/
 @Immutable
-public class IdPJWTSecurityContext implements SecurityContext{
+public class ProfileRequestJWTSecurityContext implements SecurityContext{
     
     /** The profile request context.*/
     @Nonnull private final ProfileRequestContext prc;
@@ -38,7 +38,7 @@ public class IdPJWTSecurityContext implements SecurityContext{
      * 
      * @param requestContext the profile request context
      */
-    public IdPJWTSecurityContext(@Nonnull final ProfileRequestContext requestContext) {
+    public ProfileRequestJWTSecurityContext(@Nonnull final ProfileRequestContext requestContext) {
         prc = Constraint.isNotNull(requestContext, "ProfileRequestContext can not be null");
     }
     
diff --git a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/ValidateTokenClaims.java b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/ValidateTokenClaims.java
index 825f7b9..28b8ed0 100644
--- a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/ValidateTokenClaims.java
+++ b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/ValidateTokenClaims.java
@@ -247,7 +247,7 @@ public class ValidateTokenClaims extends AbstractDuoAuthenticationAction {
         claimsVerifier.setAuthnLifetime(authnLifetime);
         
         try {
-            claimsVerifier.verify(claimsSet, new IdPJWTSecurityContext(profileRequestContext));
+            claimsVerifier.verify(claimsSet, new ProfileRequestJWTSecurityContext(profileRequestContext));
         } catch (final BadJWTException e) {
             log.error("{} Token verification failed for subject '{}'", getLogPrefix(),claimsSet.getSubject(),e);
             ActionSupport.buildEvent(profileRequestContext, AuthnEventIds.NO_CREDENTIALS);

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


More information about the commits mailing list