[jboss-cvs] JBossAS SVN: r96168 - branches/JBPAPP_5_0/console/src/main/org/jboss/console/twiddle.

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

Author: mmoyses
Date: 2009-11-09 08:39:55 -0500 (Mon, 09 Nov 2009)
New Revision: 96168

Modified:
   branches/JBPAPP_5_0/console/src/main/org/jboss/console/twiddle/Twiddle.java
Log:
JBPAPP-2873: masking passwords from twiddle log

Modified: branches/JBPAPP_5_0/console/src/main/org/jboss/console/twiddle/Twiddle.java
===================================================================
--- branches/JBPAPP_5_0/console/src/main/org/jboss/console/twiddle/Twiddle.java	2009-11-09 13:35:31 UTC (rev 96167)
+++ branches/JBPAPP_5_0/console/src/main/org/jboss/console/twiddle/Twiddle.java	2009-11-09 13:39:55 UTC (rev 96168)
@@ -404,7 +404,8 @@
    {
       for(int a = 0; a < args.length; a ++)
       {
-         log.debug("args["+a+"]="+args[a]);
+         if (!logPassword(args, a))
+            log.debug("args["+a+"]="+args[a]);
       }
       String sopts = "-:hH:u:p:c:D:s:a:q";
       LongOpt[] lopts =
@@ -542,4 +543,35 @@
            }
         }
    }
+   
+   private static boolean logPassword(final String args[], int a)
+   {
+      // check current argument
+      if (args[a].startsWith("-p") && args[a].length() > 2)
+      {
+         log.debug("args["+a+"]=-pxxxx");
+         return true;
+      }
+      else if (args[a].indexOf('=') != -1)
+      {
+         String[] split = args[a].split("=");
+         if ("--password".indexOf(split[0]) != -1)
+         {
+            log.debug("args["+a+"]="+split[0]+"=xxxx");
+            return true;
+         }
+      }
+      // check previous argument
+      try {
+         if (args[a-1].equals("-p") || (args[a-1].indexOf('=') == -1 && "--password".indexOf(args[a-1]) != -1))
+         {
+            log.debug("args["+a+"]=xxxx");
+            return true;
+         }
+      }
+      catch (IndexOutOfBoundsException ioobe)
+      {
+      }
+      return false;
+   }
 }