verify for email domain for everything but ldap / saml (#6718)

Этот коммит содержится в:
Chris
2017-06-26 11:33:48 -07:00
коммит произвёл Joram Wilander
родитель f404483834
Коммит 69ae5cef16
2 изменённых файлов: 10 добавлений и 18 удалений

Просмотреть файл

@@ -176,7 +176,7 @@ func IsFirstUserAccount() bool {
}
func CreateUser(user *model.User) (*model.User, *model.AppError) {
if !user.IsSSOUser() && !CheckUserDomain(user, utils.Cfg.TeamSettings.RestrictCreationToDomains) {
if !user.IsLDAPUser() && !user.IsSAMLUser() && !CheckUserDomain(user, utils.Cfg.TeamSettings.RestrictCreationToDomains) {
return nil, model.NewLocAppError("CreateUser", "api.user.create_user.accepted_domain.app_error", nil, "")
}
@@ -313,15 +313,13 @@ func CheckUserDomain(user *model.User, domains string) bool {
domainArray := strings.Fields(strings.TrimSpace(strings.ToLower(strings.Replace(strings.Replace(domains, "@", " ", -1), ",", " ", -1))))
matched := false
for _, d := range domainArray {
if strings.HasSuffix(strings.ToLower(user.Email), "@"+d) {
matched = true
break
return true
}
}
return matched
return false
}
// Check if the username is already used by another user. Return false if the username is invalid.