You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ca...@apache.org on 2006/09/19 00:07:33 UTC

svn commit: r447592 - /maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/aspectj/ContinuumSecurityAspect.aj

Author: carlos
Date: Mon Sep 18 15:07:32 2006
New Revision: 447592

URL: http://svn.apache.org/viewvc?view=rev&rev=447592
Log:
Exclude some methods for faster execution

Modified:
    maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/aspectj/ContinuumSecurityAspect.aj

Modified: maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/aspectj/ContinuumSecurityAspect.aj
URL: http://svn.apache.org/viewvc/maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/aspectj/ContinuumSecurityAspect.aj?view=diff&rev=447592&r1=447591&r2=447592
==============================================================================
--- maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/aspectj/ContinuumSecurityAspect.aj (original)
+++ maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/aspectj/ContinuumSecurityAspect.aj Mon Sep 18 15:07:32 2006
@@ -28,10 +28,24 @@
 public aspect ContinuumSecurityAspect extends SecurityAspect
 {
 
+    /**
+     * <p>
+     * We need to exclude remove*(int) as it conflicts with remove*( Object ).
+     * </p>
+     * <p>
+     * Some methods are excluded just for faster execution.
+     * <ul>
+     *   <li><code>getContinuum</code></li>
+     *   <li><code>getAclEventHandler</code></li>
+     * </ul>
+     * </p>
+     */
     protected pointcut securedExecution():
         target(Continuum) &&
         execution(public * *(..)) &&
         !execution(public * remove*(int)) &&
+        !execution(public * getContinuum(..)) &&
+        !execution(public * getAclEventHandler(..)) &&
         !within(ContinuumSecurityAspect);
 
 }