From e60440d7b973756faa9aa23ca0deb50977998cd9 Mon Sep 17 00:00:00 2001 From: Mathieu Baudier Date: Sat, 13 Oct 2012 08:36:44 +0000 Subject: [PATCH] Revert caching of users and roles git-svn-id: https://svn.argeo.org/commons/trunk@5601 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc --- .../security/jackrabbit/ArgeoSecurityManager.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/security/runtime/org.argeo.security.jackrabbit/src/main/java/org/argeo/security/jackrabbit/ArgeoSecurityManager.java b/security/runtime/org.argeo.security.jackrabbit/src/main/java/org/argeo/security/jackrabbit/ArgeoSecurityManager.java index 00c674580..9d3708ed3 100644 --- a/security/runtime/org.argeo.security.jackrabbit/src/main/java/org/argeo/security/jackrabbit/ArgeoSecurityManager.java +++ b/security/runtime/org.argeo.security.jackrabbit/src/main/java/org/argeo/security/jackrabbit/ArgeoSecurityManager.java @@ -46,8 +46,8 @@ public class ArgeoSecurityManager extends DefaultSecurityManager { .getLog(ArgeoSecurityManager.class); /** TODO? use a bounded buffer */ - private Map userRolesCache = Collections - .synchronizedMap(new HashMap()); +// private Map userRolesCache = Collections +// .synchronizedMap(new HashMap()); /** * Since this is called once when the session is created, we take the @@ -90,15 +90,15 @@ public class ArgeoSecurityManager extends DefaultSecurityManager { } // do not sync if not changed - if (userRolesCache.containsKey(userId) - && userRolesCache.get(userId).equals(roles.toString())) - return userId; +// if (userRolesCache.containsKey(userId) +// && userRolesCache.get(userId).equals(roles.toString())) +// return userId; // sync Spring and Jackrabbit // workspace is irrelevant here UserManager systemUm = getSystemUserManager(null); syncSpringAndJackrabbitSecurity(systemUm, authen); - userRolesCache.put(userId, roles.toString()); +// userRolesCache.put(userId, roles.toString()); } return userId; } -- 2.30.2