fix bug in AuthorizationInfo cache

This commit is contained in:
Sebastian Sdorra
2012-10-02 16:27:17 +02:00
parent 9157457206
commit 1a41802a1f

View File

@@ -270,7 +270,7 @@ public class ScmRealm extends AuthorizingRealm
{ {
User user = principals.oneByType(User.class); User user = principals.oneByType(User.class);
AuthorizationInfo info = cache.get(user.getName()); AuthorizationInfo info = cache.get(user.getId());
if (info == null) if (info == null)
{ {
@@ -282,10 +282,11 @@ public class ScmRealm extends AuthorizingRealm
GroupNames groups = principals.oneByType(GroupNames.class); GroupNames groups = principals.oneByType(GroupNames.class);
info = createAuthorizationInfo(user, groups); 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()); user.getName());
} }