You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ca...@apache.org on 2006/09/13 19:09:40 UTC
svn commit: r443032 -
/maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java
Author: carlos
Date: Wed Sep 13 10:09:39 2006
New Revision: 443032
URL: http://svn.apache.org/viewvc?view=rev&rev=443032
Log:
Store the value of locked in db
Modified:
maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java
Modified: maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java?view=diff&rev=443032&r1=443031&r2=443032
==============================================================================
--- maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java (original)
+++ maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java Wed Sep 13 10:09:39 2006
@@ -100,13 +100,7 @@
}
else
{
- // failed login. increment and test.
- if ( user.incrementFailedLoginAttempts() >= securityPolicy.getAllowedLoginAttempts() )
- {
- user.setLocked( true );
- }
-
- this.updateUser( user );
+ loginFailed( user.getUsername() );
}
return validPassword;
@@ -117,7 +111,10 @@
User user = getUser( username );
if ( user != null )
{
- user.incrementFailedLoginAttempts();
+ if ( user.incrementFailedLoginAttempts() >= securityPolicy.getAllowedLoginAttempts() )
+ {
+ user.setLocked( true );
+ }
updateUser( user );
}
}
@@ -328,7 +325,7 @@
return (Permission) userStore.addPermission( perm );
}
-
+
/**
* This implementation return empty permissions for each user.
*/