no need to try updating the user if userManagerImpl is read only

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1421158 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 10 insertions, 2 deletions

diff --git a/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaUserManagerAuthenticator.java b/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaUserManagerAuthenticator.java
index b9c79564b..767d90e7b 100644
--- a/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaUserManagerAuthenticator.java
+++ b/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaUserManagerAuthenticator.java
@@ -140,7 +140,10 @@ public class ArchivaUserManagerAuthenticator
                         if ( user.getCountFailedLoginAttempts() > 0 )
                         {
                             user.setCountFailedLoginAttempts( 0 );
-                            userManager.updateUser( user );
+                            if ( !userManager.isReadOnly() )
+                            {
+                                userManager.updateUser( user );
+                            }
                         }
 
                         return new AuthenticationResult( true, source.getPrincipal(), null );
@@ -162,11 +165,16 @@ public class ArchivaUserManagerAuthenticator
 
                     try
                     {
+
                         securityPolicy.extensionExcessiveLoginAttempts( user );
+
                     }
                     finally
                     {
-                        userManager.updateUser( user );
+                        if ( !userManager.isReadOnly() )
+                        {
+                            userManager.updateUser( user );
+                        }
                     }
 
                     //return new AuthenticationResult( false, source.getPrincipal(), null, authnResultExceptionsMap );