fixed comparison order to be null safe

cleaned up type check
pull/516/head
Justin Richer 2013-09-10 17:26:21 -04:00
parent dbdc2e777d
commit 77c0473438
1 changed files with 2 additions and 1 deletions

View File

@ -17,6 +17,7 @@
package org.mitre.openid.connect.service.impl;
import org.mitre.oauth2.model.ClientDetailsEntity;
import org.mitre.oauth2.model.ClientDetailsEntity.SubjectType;
import org.mitre.oauth2.service.ClientDetailsEntityService;
import org.mitre.openid.connect.model.UserInfo;
import org.mitre.openid.connect.repository.UserInfoRepository;
@ -74,7 +75,7 @@ public class DefaultUserInfoService implements UserInfoService {
return null;
}
if (client.getSubjectType().equals(ClientDetailsEntity.SubjectType.PAIRWISE)) {
if (SubjectType.PAIRWISE.equals(client.getSubjectType())) {
String pairwiseSub = pairwiseIdentifierService.getIdentifier(userInfo, client);
userInfo.setSub(pairwiseSub);
}