User: starksm 
  Date: 01/10/01 12:34:46

  Modified:    src/main/org/jboss/security/plugins JaasSecurityManager.java
  Log:
  Dump the LoginException even if the principal is null if trace level is
  enabled.
  
  Revision  Changes    Path
  1.14      +5 -6      
jbosssx/src/main/org/jboss/security/plugins/JaasSecurityManager.java
  
  Index: JaasSecurityManager.java
  ===================================================================
  RCS file: 
/cvsroot/jboss/jbosssx/src/main/org/jboss/security/plugins/JaasSecurityManager.java,v
  retrieving revision 1.13
  retrieving revision 1.14
  diff -u -r1.13 -r1.14
  --- JaasSecurityManager.java  2001/09/26 05:59:25     1.13
  +++ JaasSecurityManager.java  2001/10/01 19:34:46     1.14
  @@ -29,8 +29,7 @@
   import javax.security.auth.callback.PasswordCallback;
   import javax.security.auth.callback.UnsupportedCallbackException;
   
  -import org.apache.log4j.Category;
  -
  +import org.jboss.logging.Logger;
   import org.jboss.security.AppPolicy;
   import org.jboss.security.AuthenticationInfo;
   import org.jboss.security.EJBSecurityManager;
  @@ -55,7 +54,7 @@
    
    @author <a href="[EMAIL PROTECTED]">Oleg Nitz</a>
    @author [EMAIL PROTECTED]
  - @version $Revision: 1.13 $
  + @version $Revision: 1.14 $
   */
   public class JaasSecurityManager implements SubjectSecurityManager, RealmMapping
   {
  @@ -74,7 +73,7 @@
      private static ThreadLocal activeSubject = new ThreadLocal();
      /** The log4j category for the security manager domain
       */
  -   private Category log;
  +   private Logger log;
      /** The name of the domain this instance is securing. It is used as
       the appName into the SecurityPolicy.
       */
  @@ -143,7 +142,7 @@
      public JaasSecurityManager(String securityDomain)
      {
         this.securityDomain = securityDomain;
  -      this.log = Category.getInstance(getClass().getName()+"."+securityDomain);
  +      this.log = Logger.create(getClass().getName()+"."+securityDomain);
      }
   
      /** The domainCache is typically a shared object that is populated
  @@ -364,7 +363,7 @@
         catch(LoginException e)
         {
            // Don't log anonymous user failures
  -         if( principal != null && principal.getName() != null )
  +         if( principal != null && principal.getName() != null || 
log.isTraceEnabled() )
               log.debug("Login failure", e);
         }
   
  
  
  

_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to