[jboss-cvs] JBossAS SVN: r104066 - branches/JBPAPP_5_0/server/src/main/org/jboss/ejb.

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

Author: mmoyses
Date: 2010-04-16 15:03:08 -0400 (Fri, 16 Apr 2010)
New Revision: 104066

Modified:
   branches/JBPAPP_5_0/server/src/main/org/jboss/ejb/EjbModule.java
Log:
JBPAPP-4121: fix warning for unsecured MDBs

Modified: branches/JBPAPP_5_0/server/src/main/org/jboss/ejb/EjbModule.java
===================================================================
--- branches/JBPAPP_5_0/server/src/main/org/jboss/ejb/EjbModule.java	2010-04-16 18:59:56 UTC (rev 104065)
+++ branches/JBPAPP_5_0/server/src/main/org/jboss/ejb/EjbModule.java	2010-04-16 19:03:08 UTC (rev 104066)
@@ -842,7 +842,8 @@
       // Set security domain manager
       String securityDomain = bean.getApplicationMetaData().getSecurityDomain();
       // JBAS-5960: Set default security domain if there is security metadata
-      if (securityDomain == null && hasSecurityMetaData(bean))
+      boolean hasSecurityMetaData = hasSecurityMetaData(bean);
+      if (securityDomain == null && hasSecurityMetaData)
       {
          securityDomain = SecurityConstants.DEFAULT_EJB_APPLICATION_POLICY;
       }
@@ -888,8 +889,9 @@
       }
       else
       {
-         log.warn("EJB configured to bypass security. Please verify if this is intended. Bean=" + bean.getEjbName()
-               + " Deployment=" + unit.getName());
+    	 if ("".equals(securityDomain) && hasSecurityMetaData)
+    		log.warn("EJB configured to bypass security. Please verify if this is intended. Bean=" + bean.getEjbName()
+    			  + " Deployment=" + unit.getName());
       }
 
       // Load the security proxy instance if one was configured