Merge pull request 'Restored session existance check before OIDC/UMA ticket related checks' (#9) from mauro.mugnaini/threadlocal-vars-cleaner:master into master
Reviewed-on: #9
This commit is contained in:
commit
5f066da115
|
@ -71,10 +71,15 @@ public class SmartGearsPortalValve extends ValveBase {
|
||||||
|
|
||||||
_log.debug("Getting current user");
|
_log.debug("Getting current user");
|
||||||
User user = getCurrentUser(request);
|
User user = getCurrentUser(request);
|
||||||
|
// user cannot be null otherwise also the getCurrentUsername(request) returned null username in previous lines
|
||||||
_log.debug("Getting current session");
|
_log.debug("Getting current session");
|
||||||
HttpSession session = request.getSession(false);
|
HttpSession session = request.getSession(false);
|
||||||
|
if (session != null) {
|
||||||
OIDCUmaUtil.checkUMATicketAndProvideInThreadLocal(request, (HttpServletResponse) resp, user,
|
OIDCUmaUtil.checkUMATicketAndProvideInThreadLocal(request, (HttpServletResponse) resp, user,
|
||||||
session, scope);
|
session, scope);
|
||||||
|
} else {
|
||||||
|
_log.debug("Session is null, cannot continue");
|
||||||
|
}
|
||||||
|
|
||||||
//_log.trace("Security token set OK for " + username + " in scope " + scope);
|
//_log.trace("Security token set OK for " + username + " in scope " + scope);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue