From effe9559533b5a8908912db42dd0d0280c85927b Mon Sep 17 00:00:00 2001 From: Mark Janssen Date: Sun, 24 May 2015 17:53:53 +0200 Subject: [PATCH] Fix blacklist/whitelist for DynamicRegistrationClientConfigurationService ClientConfigurationService#getClientConfiguration has ServerConfiguration as parameter, not String --- .../impl/DynamicRegistrationClientConfigurationService.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/openid-connect-client/src/main/java/org/mitre/openid/connect/client/service/impl/DynamicRegistrationClientConfigurationService.java b/openid-connect-client/src/main/java/org/mitre/openid/connect/client/service/impl/DynamicRegistrationClientConfigurationService.java index 5e2319327..46489165f 100644 --- a/openid-connect-client/src/main/java/org/mitre/openid/connect/client/service/impl/DynamicRegistrationClientConfigurationService.java +++ b/openid-connect-client/src/main/java/org/mitre/openid/connect/client/service/impl/DynamicRegistrationClientConfigurationService.java @@ -76,11 +76,11 @@ public class DynamicRegistrationClientConfigurationService implements ClientConf @Override public RegisteredClient getClientConfiguration(ServerConfiguration issuer) { try { - if (!whitelist.isEmpty() && !whitelist.contains(issuer)) { + if (!whitelist.isEmpty() && !whitelist.contains(issuer.getIssuer())) { throw new AuthenticationServiceException("Whitelist was nonempty, issuer was not in whitelist: " + issuer); } - if (blacklist.contains(issuer)) { + if (blacklist.contains(issuer.getIssuer())) { throw new AuthenticationServiceException("Issuer was in blacklist: " + issuer); }