[jboss-cvs] JBossAS SVN: r57664 - branches/JEE5_TCK/ejb3/src/main/org/jboss/ejb3/security

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

Author: anil.saldhana at jboss.com
Date: 2006-10-13 14:19:20 -0400 (Fri, 13 Oct 2006)
New Revision: 57664

Modified:
   branches/JEE5_TCK/ejb3/src/main/org/jboss/ejb3/security/JaccHelper.java
Log:
EJBTHREE-745:create jacc permissions even if the SecurityDomain annotation is absent

Modified: branches/JEE5_TCK/ejb3/src/main/org/jboss/ejb3/security/JaccHelper.java
===================================================================
--- branches/JEE5_TCK/ejb3/src/main/org/jboss/ejb3/security/JaccHelper.java	2006-10-13 17:41:28 UTC (rev 57663)
+++ branches/JEE5_TCK/ejb3/src/main/org/jboss/ejb3/security/JaccHelper.java	2006-10-13 18:19:20 UTC (rev 57664)
@@ -46,7 +46,9 @@
 import org.jboss.logging.Logger;
 
 /**
+ * JACC Helper class that created permissions as well as done the checks
  * @author <a href="mailto:kabir.khan at jboss.org">Kabir Khan</a>
+ * @author Anil.Saldhana at jboss.com
  * @version $Revision$
  */
 public class JaccHelper
@@ -137,15 +139,8 @@
 
    private static void addPermissions(EJBContainer container, PolicyConfiguration pc)
    {
-      SecurityDomain sd = (SecurityDomain) container.resolveAnnotation(SecurityDomain.class);
+      SecurityDomain sd = (SecurityDomain) container.resolveAnnotation(SecurityDomain.class); 
 
-      if (sd == null)
-      {
-         log.debug(container.getEjbName() + " has no @SecurityDomain - skipping JACC configuration");
-         return;
-      }
-      log.debug(container.getEjbName() + " has @SecurityDomain - peforming JACC configuration");
-
       PermitAll beanUnchecked = (PermitAll) container.resolveAnnotation(PermitAll.class);
       RolesAllowed beanPermissions = (RolesAllowed) container.resolveAnnotation(RolesAllowed.class);