[java-idp-plugin-oidc-rp] branch main updated: Remove unused classes

Phil Smart philip.smart at jisc.ac.uk
Mon Apr 24 13:26:18 UTC 2023


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

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

View the commit online:
http://git.shibboleth.net/view/?p=java-idp-plugin-oidc-rp.git;a=commit;h=0ac9c07405ddd03e690923fa88764ada94de2af1

The following commit(s) were added to refs/heads/main by this push:
     new 0ac9c07  Remove unused classes
0ac9c07 is described below

commit 0ac9c07405ddd03e690923fa88764ada94de2af1
Author: Phil Smart <philip.smart at jisc.ac.uk>
AuthorDate: Mon Apr 24 14:26:13 2023 +0100

    Remove unused classes
---
 .../UserInfoEncryptedJWTResponseTypeCondition.java | 45 ----------------------
 .../UserInfoSignedJWTResponseTypeCondition.java    | 44 ---------------------
 .../OIDCRPFlowFromAuthenticationResponseTest.java  |  2 +-
 3 files changed, 1 insertion(+), 90 deletions(-)

diff --git a/idp-oidc-rp-api/src/main/java/net/shibboleth/idp/plugin/authn/oidc/rp/messaging/context/logic/UserInfoEncryptedJWTResponseTypeCondition.java b/idp-oidc-rp-api/src/main/java/net/shibboleth/idp/plugin/authn/oidc/rp/messaging/context/logic/UserInfoEncryptedJWTResponseTypeCondition.java
deleted file mode 100644
index 15507ac..0000000
--- a/idp-oidc-rp-api/src/main/java/net/shibboleth/idp/plugin/authn/oidc/rp/messaging/context/logic/UserInfoEncryptedJWTResponseTypeCondition.java
+++ /dev/null
@@ -1,45 +0,0 @@
-/*
- * Licensed to the University Corporation for Advanced Internet Development,
- * Inc. (UCAID) under one or more contributor license agreements.  See the
- * NOTICE file distributed with this work for additional information regarding
- * copyright ownership. The UCAID licenses this file to You under the Apache
- * License, Version 2.0 (the "License"); you may not use this file except in
- * compliance with the License.  You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package net.shibboleth.idp.plugin.authn.oidc.rp.messaging.context.logic;
-
-import javax.annotation.Nonnull;
-
-import org.opensaml.profile.context.ProfileRequestContext;
-
-import com.nimbusds.jose.JWEAlgorithm;
-import com.nimbusds.jwt.EncryptedJWT;
-
-import net.shibboleth.idp.plugin.authn.oidc.rp.context.UserInfoResponseContext;
-
-/**
- * Return true if the UserInfo JWT response was an encrypted JWT type.
- */
-public class UserInfoEncryptedJWTResponseTypeCondition extends AbstractUserInfoResponseTypeCondition {
-
-    @Override
-    protected boolean doTest(@Nonnull final ProfileRequestContext prc, 
-            @Nonnull final UserInfoResponseContext context) {
-        if (context.getUserInfo() == null || context.getUserInfo().getUserInfoJWT() == null ||
-                context.getUserInfo().getUserInfoJWT().getHeader() == null) {
-            return false;
-        }
-        return (JWEAlgorithm.Family.SYMMETRIC.contains(context.getUserInfo().getUserInfoJWT().getHeader()
-                .getAlgorithm()) || JWEAlgorithm.Family.ASYMMETRIC.contains(context.getUserInfo().getUserInfoJWT()
-                        .getHeader().getAlgorithm())) && context.getUserInfo().getUserInfoJWT() instanceof EncryptedJWT;
-    }
-}
\ No newline at end of file
diff --git a/idp-oidc-rp-api/src/main/java/net/shibboleth/idp/plugin/authn/oidc/rp/messaging/context/logic/UserInfoSignedJWTResponseTypeCondition.java b/idp-oidc-rp-api/src/main/java/net/shibboleth/idp/plugin/authn/oidc/rp/messaging/context/logic/UserInfoSignedJWTResponseTypeCondition.java
deleted file mode 100644
index 0596f65..0000000
--- a/idp-oidc-rp-api/src/main/java/net/shibboleth/idp/plugin/authn/oidc/rp/messaging/context/logic/UserInfoSignedJWTResponseTypeCondition.java
+++ /dev/null
@@ -1,44 +0,0 @@
-/*
- * Licensed to the University Corporation for Advanced Internet Development,
- * Inc. (UCAID) under one or more contributor license agreements.  See the
- * NOTICE file distributed with this work for additional information regarding
- * copyright ownership. The UCAID licenses this file to You under the Apache
- * License, Version 2.0 (the "License"); you may not use this file except in
- * compliance with the License.  You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package net.shibboleth.idp.plugin.authn.oidc.rp.messaging.context.logic;
-
-import javax.annotation.Nonnull;
-
-import org.opensaml.profile.context.ProfileRequestContext;
-
-import com.nimbusds.jose.JWSAlgorithm;
-import com.nimbusds.jwt.SignedJWT;
-
-import net.shibboleth.idp.plugin.authn.oidc.rp.context.UserInfoResponseContext;
-
-/**
- * Return true if the UserInfo JWT response was a signed JWT type.
- */
-public class UserInfoSignedJWTResponseTypeCondition extends AbstractUserInfoResponseTypeCondition {
-
-    @Override
-    protected boolean doTest(@Nonnull final ProfileRequestContext prc, 
-            @Nonnull final UserInfoResponseContext context) {
-        if (context.getUserInfo() == null || context.getUserInfo().getUserInfoJWT() == null) {
-            return false;
-        }
-        
-        return JWSAlgorithm.Family.SIGNATURE.contains(context.getUserInfo().getUserInfoJWT().getHeader()
-                .getAlgorithm()) && context.getUserInfo().getUserInfoJWT() instanceof SignedJWT;
-    }
-}
diff --git a/idp-oidc-rp-impl/src/test/java/net/shibboleth/idp/plugin/authn/oidc/rp/impl/OIDCRPFlowFromAuthenticationResponseTest.java b/idp-oidc-rp-impl/src/test/java/net/shibboleth/idp/plugin/authn/oidc/rp/impl/OIDCRPFlowFromAuthenticationResponseTest.java
index 315df14..b45ef42 100644
--- a/idp-oidc-rp-impl/src/test/java/net/shibboleth/idp/plugin/authn/oidc/rp/impl/OIDCRPFlowFromAuthenticationResponseTest.java
+++ b/idp-oidc-rp-impl/src/test/java/net/shibboleth/idp/plugin/authn/oidc/rp/impl/OIDCRPFlowFromAuthenticationResponseTest.java
@@ -183,7 +183,7 @@ public class OIDCRPFlowFromAuthenticationResponseTest extends OIDCRPFlowTest {
     }
     
     @Test 
-    public void test_IDTokenHS256_UserInfo_JWT_NotSigned() throws Exception {
+    public void test_IDTokenHS256_UserInfoJWTNotSigned() throws Exception {
         
         basicSetup();
         

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


More information about the commits mailing list