add Trusted-Certificates method to AbstractCredentialProvider

add jUnit tests for AbstractCredentialProvider
change method names in AbstractCredentialProvider

1 files changed, 8 insertions, 27 deletions

diff --git a/eaaf_modules/eaaf_module_pvp2_idp/src/main/java/at/gv/egiz/eaaf/modules/pvp2/idp/impl/builder/AuthResponseBuilder.java b/eaaf_modules/eaaf_module_pvp2_idp/src/main/java/at/gv/egiz/eaaf/modules/pvp2/idp/impl/builder/AuthResponseBuilder.java
index 55e3e8b4..565f28fb 100644
--- a/eaaf_modules/eaaf_module_pvp2_idp/src/main/java/at/gv/egiz/eaaf/modules/pvp2/idp/impl/builder/AuthResponseBuilder.java
+++ b/eaaf_modules/eaaf_module_pvp2_idp/src/main/java/at/gv/egiz/eaaf/modules/pvp2/idp/impl/builder/AuthResponseBuilder.java
@@ -19,9 +19,6 @@
 
 package at.gv.egiz.eaaf.modules.pvp2.idp.impl.builder;
 
-import java.security.PublicKey;
-import java.security.interfaces.ECPublicKey;
-import java.security.interfaces.RSAPublicKey;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -134,7 +131,14 @@ public class AuthResponseBuilder {
       X509Credential encryptionCredentials, IConfiguration authConfig)
       throws InvalidAssertionEncryptionException {
     try {
-      final String keyEncAlg = selectKeyEncryptionAlgorithm(encryptionCredentials, authConfig);
+      final String keyEncAlg = Saml2Utils.getKeyOperationAlgorithmFromCredential(
+          encryptionCredentials,
+          authConfig.getBasicConfiguration(
+              PvpConstants.CONFIG_PROP_SEC_ENCRYPTION_KEY_RSA_ALG,
+              PvpConstants.DEFAULT_ASYM_ENCRYPTION_METHODE_RSA),
+          authConfig.getBasicConfiguration(
+              PvpConstants.CONFIG_PROP_SEC_ENCRYPTION_KEY_EC_ALG,
+              PvpConstants.DEFAULT_ASYM_ENCRYPTION_METHODE_EC));
 
       final DataEncryptionParameters dataEncParams = new DataEncryptionParameters();
       dataEncParams.setAlgorithm(authConfig.getBasicConfiguration(
@@ -164,29 +168,6 @@ public class AuthResponseBuilder {
 
   }
 
-  private static String selectKeyEncryptionAlgorithm(X509Credential encryptionCredentials,
-      IConfiguration authConfig) throws SamlSigningException {
-    final PublicKey privatekey = encryptionCredentials.getPublicKey();
-    if (privatekey instanceof RSAPublicKey) {
-      return authConfig.getBasicConfiguration(
-          PvpConstants.CONFIG_PROP_SEC_ENCRYPTION_KEY_RSA_ALG,
-          PvpConstants.DEFAULT_ASYM_ENCRYPTION_METHODE_RSA);
-
-    } else if (privatekey instanceof ECPublicKey) {
-      return authConfig.getBasicConfiguration(
-          PvpConstants.CONFIG_PROP_SEC_ENCRYPTION_KEY_EC_ALG,
-          PvpConstants.DEFAULT_ASYM_ENCRYPTION_METHODE_EC);
-
-    } else {
-      log.warn("Could NOT evaluate the Private-Key type from " + encryptionCredentials.getEntityId()
-          + " credential.");
-      throw new SamlSigningException("internal.pvp.97",
-          new Object[] { encryptionCredentials.getEntityId(), privatekey.getClass().getName() });
-
-    }
-
-  }
-
   private static X509Credential resolveEncryptionCredential(RequestAbstractType req,
       IPvp2MetadataProvider metadataProvider) throws InvalidAssertionEncryptionException {
     try {