You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by at...@apache.org on 2011/05/11 17:28:53 UTC

svn commit: r1101923 - /portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/impl/PermissionManagerImpl.java

Author: ate
Date: Wed May 11 15:28:53 2011
New Revision: 1101923

URL: http://svn.apache.org/viewvc?rev=1101923&view=rev
Log:
JS2-589: PermissionManagerImpl use of ThreadLocal for caching causes inconsistent results
- fixed by registering the ThreadLocal (state) cleanup operation through the new ServletRequestCleanupService (JS2-1353)

Modified:
    portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/impl/PermissionManagerImpl.java

Modified: portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/impl/PermissionManagerImpl.java
URL: http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/impl/PermissionManagerImpl.java?rev=1101923&r1=1101922&r2=1101923&view=diff
==============================================================================
--- portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/impl/PermissionManagerImpl.java (original)
+++ portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/impl/PermissionManagerImpl.java Wed May 11 15:28:53 2011
@@ -34,6 +34,7 @@ import org.apache.jetspeed.security.spi.
 import org.apache.jetspeed.security.spi.PersistentJetspeedPermission;
 import org.apache.jetspeed.security.spi.impl.BaseJetspeedPermission;
 import org.apache.jetspeed.security.spi.impl.JetspeedPermissionFactory;
+import org.apache.jetspeed.util.ServletRequestThreadLocalCleanupCallback;
 
 /**
  * @version $Id$
@@ -63,6 +64,7 @@ public class PermissionManagerImpl imple
         {
             map = new HashMap<Long,Permissions>();
             permissionsCache.set(map);
+            new ServletRequestThreadLocalCleanupCallback(permissionsCache);            
         }
         return map;
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org