[jboss-cvs] Picketbox SVN: r149 - in trunk/security-spi/spi/src: test/java/org/jboss/test/security/otp and 1 other directory.

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

Author: anil.saldhana at jboss.com
Date: 2011-01-12 10:34:54 -0500 (Wed, 12 Jan 2011)
New Revision: 149

Modified:
   trunk/security-spi/spi/src/main/java/org/jboss/security/otp/TimeBasedOTPUtil.java
   trunk/security-spi/spi/src/test/java/org/jboss/test/security/otp/TimeBasedOTPUnitTestCase.java
Log:
SECURITY-550: correct the time

Modified: trunk/security-spi/spi/src/main/java/org/jboss/security/otp/TimeBasedOTPUtil.java
===================================================================
--- trunk/security-spi/spi/src/main/java/org/jboss/security/otp/TimeBasedOTPUtil.java	2011-01-12 15:30:10 UTC (rev 148)
+++ trunk/security-spi/spi/src/main/java/org/jboss/security/otp/TimeBasedOTPUtil.java	2011-01-12 15:34:54 UTC (rev 149)
@@ -48,12 +48,11 @@
      
      String generatedTOTP = TimeBasedOTP.generateTOTP( new String( secret ) , numDigits );
      boolean result =  generatedTOTP.equals( submittedOTP );
+
+     long timeInMilis = currentDateTime.getTimeInMillis();
      
      if( !result )
-     {
-        //Step back time interval
-        long timeInMilis = currentDateTime.getTimeInMillis();
-        
+     {   
         timeInMilis -= TIME_INTERVAL;
         
         generatedTOTP = TimeBasedOTP.generateTOTP( new String( secret ) , "" + timeInMilis, numDigits );
@@ -62,8 +61,6 @@
      
      if( !result )
      {
-        //Step ahead time interval
-        long timeInMilis = currentDateTime.getTimeInMillis();
         timeInMilis += TIME_INTERVAL;
         generatedTOTP = TimeBasedOTP.generateTOTP( new String( secret ) , "" + timeInMilis, numDigits );
         result =  generatedTOTP.equals( submittedOTP );

Modified: trunk/security-spi/spi/src/test/java/org/jboss/test/security/otp/TimeBasedOTPUnitTestCase.java
===================================================================
--- trunk/security-spi/spi/src/test/java/org/jboss/test/security/otp/TimeBasedOTPUnitTestCase.java	2011-01-12 15:30:10 UTC (rev 148)
+++ trunk/security-spi/spi/src/test/java/org/jboss/test/security/otp/TimeBasedOTPUnitTestCase.java	2011-01-12 15:34:54 UTC (rev 149)
@@ -87,7 +87,7 @@
    
    int NUMBER_OF_DIGITS = 8;
    
-   int SLEEP_TIME = 10;
+   int SLEEP_TIME = 2;
    
    
    @Test