[jboss-cvs] JBossAS SVN: r70622 - projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/mapping/providers.

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

Author: anil.saldhana at jboss.com
Date: 2008-03-10 10:48:25 -0400 (Mon, 10 Mar 2008)
New Revision: 70622

Modified:
   projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/mapping/providers/DeploymentRolesMappingProvider.java
Log:
relocate the trace statement

Modified: projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/mapping/providers/DeploymentRolesMappingProvider.java
===================================================================
--- projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/mapping/providers/DeploymentRolesMappingProvider.java	2008-03-10 14:17:23 UTC (rev 70621)
+++ projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/mapping/providers/DeploymentRolesMappingProvider.java	2008-03-10 14:48:25 UTC (rev 70622)
@@ -74,11 +74,13 @@
       //Obtain the principal to roles mapping
       Principal principal = (Principal) map.get(SecurityConstants.PRINCIPAL_IDENTIFIER);
       Map<String,Set<String>> principalRolesMap = (Map<String,Set<String>>)map.get(SecurityConstants.DEPLOYMENT_PRINCIPAL_ROLES_MAP);
-      if(trace)
-         log.trace("Principal="+principal+":principalRolesMap="+principalRolesMap);
       
       Set<Principal> subjectPrincipals = (Set<Principal>) map.get(SecurityConstants.PRINCIPALS_SET_IDENTIFIER);
-      
+      if(trace)
+      {
+         log.trace("Principal="+principal+":principalRolesMap="+principalRolesMap+":");
+         log.trace("subjectPrincipals="+subjectPrincipals);
+      }
       if(principalRolesMap == null || principalRolesMap.isEmpty())
       {
          result.setMappedObject(mappedObject);