catch the case where an non-email is provided (#152)

DeviaVir · Aug 27, 2020 · 2be2a54 · 2be2a54
1 parent 75277e5
commit 2be2a54
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/gsuite/utils.go b/gsuite/utils.go
@@ -187,6 +187,7 @@ func validateEmail(v interface{}, k string) (warnings []string, errors []error)
 	if err != nil {
 		errors = append(errors,
 			fmt.Errorf("unable to parse email address %s", email))
+		return
 	}
 
 	if e.Name != "" {

diff --git a/gsuite/utils_test.go b/gsuite/utils_test.go
@@ -14,6 +14,7 @@ func TestValidateEmail(t *testing.T) {
 		{"Alice <[email protected]>", false},
 		{"much-much-much-much-much-much-much-much-much-much-much-much-too-long@domain.com", false},
 		{"\"some@much-much-much-much-much-much-much-much-much-much-much-much-too-long\"@domain.com", false},
+		{"not an email address", false},
 	}
 
 	for _, testCase := range testCases {