From 1a41802a1f8e9cc319d73bee8fee9848619ae0c8 Mon Sep 17 00:00:00 2001 From: Sebastian Sdorra Date: Tue, 2 Oct 2012 16:27:17 +0200 Subject: [PATCH] fix bug in AuthorizationInfo cache --- scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java b/scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java index 6589c764d5..6aeecf8dae 100644 --- a/scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java +++ b/scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java @@ -270,7 +270,7 @@ public class ScmRealm extends AuthorizingRealm { User user = principals.oneByType(User.class); - AuthorizationInfo info = cache.get(user.getName()); + AuthorizationInfo info = cache.get(user.getId()); if (info == null) { @@ -282,10 +282,11 @@ public class ScmRealm extends AuthorizingRealm GroupNames groups = principals.oneByType(GroupNames.class); info = createAuthorizationInfo(user, groups); + cache.put(user.getId(), info); } - else if (logger.isDebugEnabled()) + else if (logger.isTraceEnabled()) { - logger.debug("retrieve AuthorizationInfo for user {} from cache", + logger.trace("retrieve AuthorizationInfo for user {} from cache", user.getName()); }