[Openutils-svn] SF.net SVN: openutils: [642] trunk/openutils-hibernate-security/src/main/ java/it/openutils/hibernate/security/aop/AOPSecurity.java

[<fc...@us...>]

Revision: 642
          http://openutils.svn.sourceforge.net/openutils/?rev=642&view=rev
Author:   fcarone
Date:     2008-02-19 09:42:20 -0800 (Tue, 19 Feb 2008)

Log Message:
-----------
conservative logging

Modified Paths:
--------------
    trunk/openutils-hibernate-security/src/main/java/it/openutils/hibernate/security/aop/AOPSecurity.java

Modified: trunk/openutils-hibernate-security/src/main/java/it/openutils/hibernate/security/aop/AOPSecurity.java
===================================================================
--- trunk/openutils-hibernate-security/src/main/java/it/openutils/hibernate/security/aop/AOPSecurity.java	2008-02-19 16:01:18 UTC (rev 641)
+++ trunk/openutils-hibernate-security/src/main/java/it/openutils/hibernate/security/aop/AOPSecurity.java	2008-02-19 17:42:20 UTC (rev 642)
@@ -97,17 +97,19 @@
 
         if (rules.isEmpty())
         {
-            String grantedRoles = StringUtils.EMPTY;
-            for (int i = 0; i < authorities.length; i++)
+            if (log.isWarnEnabled())
             {
-                grantedRoles += authorities[i].getAuthority() + " ";
+                String grantedRoles = StringUtils.EMPTY;
+                for (int i = 0; i < authorities.length; i++)
+                {
+                    grantedRoles += authorities[i].getAuthority() + " ";
+                }
+                log.warn("Access is denied on " + entity + ", for user {} with roles {}", SecurityContextHolder
+                    .getContext()
+                    .getAuthentication()
+                    .getPrincipal()
+                    .toString(), grantedRoles);
             }
-            log.warn("Access is denied on " + entity + ", for user {} with roles {}", SecurityContextHolder
-                .getContext()
-                .getAuthentication()
-                .getPrincipal()
-                .toString(), grantedRoles);
-
             throw new SecurityException("Access denied");
         }
 
@@ -142,7 +144,6 @@
         this.securedDAOs = securedDAOs;
     }
 
-
     /**
      * Sets the enabled.
      * @param enabled the enabled to set


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.