github
/
OpenID-Connect-Java-Spring-Server
mirror of https://github.com/mitreid-connect/OpenID-Connect-Java-Spring-Server
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Fix blacklist/whitelist for DynamicRegistrationClientConfigurationService 

ClientConfigurationService#getClientConfiguration has ServerConfiguration as parameter, not String
pull/822/merge
Mark Janssen 2015-05-24 17:53:53 +02:00 committed by Justin Richer
parent caf85b990d
commit effe955953
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
openid-connect-client/src/main/java/org/mitre/openid/connect/client/service/impl/DynamicRegistrationClientConfigurationService.java
View File

@ -76,11 +76,11 @@ public class DynamicRegistrationClientConfigurationService implements ClientConf
@Override @Override
public RegisteredClient getClientConfiguration(ServerConfiguration issuer) { public RegisteredClient getClientConfiguration(ServerConfiguration issuer) {
try { 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); 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); throw new AuthenticationServiceException("Issuer was in blacklist: " + issuer);
} }