Revert caching of users and roles

[lgpl/argeo-commons.git] / security / runtime / org.argeo.security.jackrabbit / src / main / java / org / argeo / security / jackrabbit / ArgeoSecurityManager.java

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 00c674580076d7bcbf7d6bdef5b2a343fc47ca6e..9d3708ed3eaebdcaa4804c8ce00c92f69399dc52 100644 (file)
--- 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<String, String> userRolesCache = Collections
-                       .synchronizedMap(new HashMap<String, String>());
+//     private Map<String, String> userRolesCache = Collections
+//                     .synchronizedMap(new HashMap<String, String>());
 
        /**
         * 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;
        }