svn commit: r943446 - /incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/concurrent/DynamicConcurrentPolicyProvider.java

[pe...@apache.org]

Author: peter_firmstone
Date: Wed May 12 11:09:44 2010
New Revision: 943446

URL: http://svn.apache.org/viewvc?rev=943446&view=rev
Log:
River-323 Just some more refactoring still experiencing failled tests

This will break some tests causing a failled Hudson build, however I think it best to get the code out there, so I can get some assistance.

By default the qa tests now utilise the ConcurrentDynamicPolicyProvider and DynamicPolicyProvider uses a pluggable SPI.

Modified:
    incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/concurrent/DynamicConcurrentPolicyProvider.java

Modified: incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/concurrent/DynamicConcurrentPolicyProvider.java
URL: http://svn.apache.org/viewvc/incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/concurrent/DynamicConcurrentPolicyProvider.java?rev=943446&r1=943445&r2=943446&view=diff
==============================================================================
--- incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/concurrent/DynamicConcurrentPolicyProvider.java (original)
+++ incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/concurrent/DynamicConcurrentPolicyProvider.java Wed May 12 11:09:44 2010
@@ -140,7 +140,7 @@ public class DynamicConcurrentPolicyProv
         basePolicyIsDynamic = false;
         revokeable = true;
         logger = Logger.getLogger("net.jini.security.policy");
-        loggable = logger.isLoggable(Level.INFO);
+        loggable = logger.isLoggable(Level.FINEST);
     }
     
     /**
@@ -319,7 +319,7 @@ public class DynamicConcurrentPolicyProv
         if (basePolicy.implies(domain, permission)) return true;
         if (basePolicyIsDynamic) return false;
         if (loggable){
-            logger.log(Level.INFO, domain + permission.toString() + 
+            logger.log(Level.FINEST, domain + permission.toString() + 
                     ": Base policy is not dynamic and returned false" );
         }
         // If it doesn't then we should check for dynamic grants
@@ -327,7 +327,7 @@ public class DynamicConcurrentPolicyProv
         CodeSource cs = ( domain == null ? null : domain.getCodeSource());
         Collection<Permission> dynamicallyGrantedPermissions = getGrants(domain, cs, pals);
         if (loggable) {
-            logger.log(Level.INFO, "Grants: " + dynamicallyGrantedPermissions.toString());
+            logger.log(Level.FINEST, "Grants: " + dynamicallyGrantedPermissions.toString());
         }
         if (dynamicallyGrantedPermissions.isEmpty()) return false;
         // Operation starts to get expensive
@@ -352,7 +352,7 @@ public class DynamicConcurrentPolicyProv
         // If we get refreshed the cache could be empty, which is more pedantic
         // however the result may still be true so we'll return it anyway.
         if (loggable) {
-            logger.log(Level.INFO, "PermissionCollection: " + pc.toString());
+            logger.log(Level.FINEST, "PermissionCollection: " + pc.toString());
         }
         return pc.implies(permission);
     }
@@ -422,7 +422,7 @@ public class DynamicConcurrentPolicyProv
         }
         PolicyEntry pe = new PolicyEntry(domain, pal, perm);
         if (loggable){
-            logger.log(Level.INFO, "Granting: " + pe.toString());
+            logger.log(Level.FINEST, "Granting: " + pe.toString());
         }
         try {
             wl.lock();