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

made user info interceptor more null safe for client side

pull/607/head
Justin Richer 2014-05-29 22:15:30 -04:00
parent 3e4aae6c8a
commit 257312d5da
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
openid-connect-common/src/main/java/org/mitre/openid/connect/web/UserInfoInterceptor.java
View File

@ -82,8 +82,13 @@ public class UserInfoInterceptor extends HandlerInterceptorAdapter {
if (p instanceof OIDCAuthenticationToken) { if (p instanceof OIDCAuthenticationToken) {
// if they're logging into this server from a remote OIDC server, pass through their user info // if they're logging into this server from a remote OIDC server, pass through their user info
OIDCAuthenticationToken oidc = (OIDCAuthenticationToken) p; OIDCAuthenticationToken oidc = (OIDCAuthenticationToken) p;
modelAndView.addObject("userInfo", oidc.getUserInfo()); if (oidc.getUserInfo() != null) {
modelAndView.addObject("userInfoJson", oidc.getUserInfo().toJson()); modelAndView.addObject("userInfo", oidc.getUserInfo());
modelAndView.addObject("userInfoJson", oidc.getUserInfo().toJson());
} else {
modelAndView.addObject("userInfo", null);
modelAndView.addObject("userInfoJson", "null");
}
} else { } else {
// don't bother checking if we don't have a principal or a userInfoService to work with // don't bother checking if we don't have a principal or a userInfoService to work with
if (p != null && p.getName() != null && userInfoService != null) { if (p != null && p.getName() != null && userInfoService != null) {