summaryrefslogtreecommitdiffstats
path: root/model/user.go
diff options
context:
space:
mode:
authorHarrison Healey <harrisonmhealey@gmail.com>2018-08-01 15:18:14 -0400
committerGitHub <noreply@github.com>2018-08-01 15:18:14 -0400
commitc34b30a6e7fb707ae12e78a51e5bd776e5ca85ed (patch)
tree54a3a5d715fb0cf337c98a47efc4a75f7b23a791 /model/user.go
parenta8cc646eede863e35eff3017f4472ec6f9ad01a4 (diff)
downloadchat-c34b30a6e7fb707ae12e78a51e5bd776e5ca85ed.tar.gz
chat-c34b30a6e7fb707ae12e78a51e5bd776e5ca85ed.tar.bz2
chat-c34b30a6e7fb707ae12e78a51e5bd776e5ca85ed.zip
MM-11521/MM-11522 Fix being able to create users with invalid emails through API (#9199)
* MM-11522 Fix being able to create users with invalid emails through API * Ensure store tests are using valid emails * Add missing license header * Remove invalid test case * Fix TestUpdateOAuthUserAttrs
Diffstat (limited to 'model/user.go')
-rw-r--r--model/user.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/model/user.go b/model/user.go
index 3e99c6fa4..c7ba6b9cb 100644
--- a/model/user.go
+++ b/model/user.go
@@ -132,7 +132,7 @@ func (u *User) IsValid() *AppError {
return InvalidUserError("username", u.Id)
}
- if len(u.Email) > USER_EMAIL_MAX_LENGTH || len(u.Email) == 0 {
+ if len(u.Email) > USER_EMAIL_MAX_LENGTH || len(u.Email) == 0 || !IsValidEmail(u.Email) {
return InvalidUserError("email", u.Id)
}