From 7f5f34eddcfa3f0f1d70976fd0a3de164e42c06f Mon Sep 17 00:00:00 2001
From: Sebastian Sdorra <sebastian.sdorra@cloudogu.com>
Date: Thu, 4 Oct 2012 11:18:10 +0200
Subject: [PATCH] do not store non valid users

---
 scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java b/scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java
index d65fec592f..479087d432 100644
--- a/scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java
+++ b/scm-webapp/src/main/java/sonia/scm/security/ScmRealm.java
@@ -329,14 +329,20 @@ public class ScmRealm extends AuthorizingRealm
       }
 
       // create new user
-      else
+      else if (user.isValid())
       {
         user.setCreationDate(System.currentTimeMillis());
+
         // TODO find a better way
         UserEventHack.fireEvent(userManager, user, HandlerEvent.BEFORE_CREATE);
         userDAO.add(user);
         UserEventHack.fireEvent(userManager, user, HandlerEvent.CREATE);
       }
+      else if (logger.isErrorEnabled())
+      {
+        logger.error("could not create user {}, beacause it is not valid",
+          user.getName());
+      }
 
       if (user.isActive())
       {