Fixes issue with LDAP inserting users with blank names.

diff --git a/models/repo.go b/models/repo.go
index cc4b53b0f299539945be92c4139bead0b2db6423..f144be5a3fd5018047cadacfe3258e526739ca97 100644 (file)
--- a/models/repo.go
+++ b/models/repo.go
@@ -40,6 +40,7 @@ var (
        ErrRepoFileNotLoaded = errors.New("Repository file not loaded")
        ErrMirrorNotExist    = errors.New("Mirror does not exist")
        ErrInvalidReference  = errors.New("Invalid reference specified")
+       ErrNameEmpty         = errors.New("Name is empty")
 )
 
 var (
@@ -259,7 +260,11 @@ var (
 
 // IsUsableName checks if name is reserved or pattern of name is not allowed.
 func IsUsableName(name string) error {
-       name = strings.ToLower(name)
+       name = strings.TrimSpace(strings.ToLower(name))
+       if utf8.RuneCountInString(name) == 0 {
+               return ErrNameEmpty
+       }
+
        for i := range reservedNames {
                if name == reservedNames[i] {
                        return ErrNameReserved{name}