[jboss-cvs] Picketlink SVN: r1397 - federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/sts/registry.

[jboss-cvs-commits at lists.jboss.org]

Author: anil.saldhana at jboss.com
Date: 2012-02-15 15:03:53 -0500 (Wed, 15 Feb 2012)
New Revision: 1397

Modified:
   federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/sts/registry/DefaultTokenRegistry.java
Log:
PLFED-250: make DefaultTokenRegistry use a concurrenthashmap

Modified: federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/sts/registry/DefaultTokenRegistry.java
===================================================================
--- federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/sts/registry/DefaultTokenRegistry.java	2012-02-15 20:01:47 UTC (rev 1396)
+++ federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/sts/registry/DefaultTokenRegistry.java	2012-02-15 20:03:53 UTC (rev 1397)
@@ -22,8 +22,8 @@
 package org.picketlink.identity.federation.core.sts.registry;
 
 import java.io.IOException;
-import java.util.HashMap;
 import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 
 import org.picketlink.identity.federation.core.sts.PicketLinkCoreSTS;
 
@@ -33,19 +33,19 @@
  * @since Jan 4, 2011
  */
 public class DefaultTokenRegistry implements SecurityTokenRegistry
-{ 
-   protected Map<String, Object> tokens = new HashMap<String,Object>();
-   
+{
+   protected Map<String, Object> tokens = new ConcurrentHashMap<String, Object>();
+
    /**
     * @see org.picketlink.identity.federation.core.sts.registry.SecurityTokenRegistry#addToken(java.lang.String, java.lang.Object)
     */
    public void addToken(String tokenID, Object token) throws IOException
    {
       SecurityManager sm = System.getSecurityManager();
-      if( sm != null )
-         sm.checkPermission( PicketLinkCoreSTS.rte ); 
-      
-      tokens.put( tokenID, token );
+      if (sm != null)
+         sm.checkPermission(PicketLinkCoreSTS.rte);
+
+      tokens.put(tokenID, token);
    }
 
    /**
@@ -54,20 +54,20 @@
    public Object getToken(String tokenID)
    {
       SecurityManager sm = System.getSecurityManager();
-      if( sm != null )
-         sm.checkPermission( PicketLinkCoreSTS.rte ); 
-      
-      return tokens.get( tokenID );
+      if (sm != null)
+         sm.checkPermission(PicketLinkCoreSTS.rte);
+
+      return tokens.get(tokenID);
    }
 
    /**
     * @see org.picketlink.identity.federation.core.sts.registry.SecurityTokenRegistry#removeToken(java.lang.String)
     */
    public void removeToken(String tokenID) throws IOException
-   { 
+   {
       SecurityManager sm = System.getSecurityManager();
-      if( sm != null )
-         sm.checkPermission( PicketLinkCoreSTS.rte ); 
-      tokens.remove( tokenID );
+      if (sm != null)
+         sm.checkPermission(PicketLinkCoreSTS.rte);
+      tokens.remove(tokenID);
    }
 }
\ No newline at end of file