Browse Source

Apply DefaultUserIsRestricted in CreateUser

pull/19310/head
Jimmy Praet 3 years ago
parent
commit
9020faa3ce
  1. 1
      models/user/user.go
  2. 9
      routers/web/auth/auth.go
  3. 15
      routers/web/auth/oauth.go

1
models/user/user.go

@ -637,6 +637,7 @@ func CreateUser(u *User, overwriteDefault ...*CreateUserOverwriteOptions) (err e
u.EmailNotificationsPreference = setting.Admin.DefaultEmailNotification
u.MaxRepoCreation = -1
u.Theme = setting.UI.DefaultTheme
u.IsRestricted = u.IsRestricted || setting.Service.DefaultUserIsRestricted
// overwrite defaults if set
if len(overwriteDefault) != 0 && overwriteDefault[0] != nil {

9
routers/web/auth/auth.go

@ -507,11 +507,10 @@ func SignUpPost(ctx *context.Context) {
}
u := &user_model.User{
Name: form.UserName,
Email: form.Email,
Passwd: form.Password,
IsActive: !(setting.Service.RegisterEmailConfirm || setting.Service.RegisterManualConfirm),
IsRestricted: setting.Service.DefaultUserIsRestricted,
Name: form.UserName,
Email: form.Email,
Passwd: form.Password,
IsActive: !(setting.Service.RegisterEmailConfirm || setting.Service.RegisterManualConfirm),
}
if !createAndHandleCreatedUser(ctx, tplSignUp, form, u, nil, false) {

15
routers/web/auth/oauth.go

@ -867,14 +867,13 @@ func SignInOAuthCallback(ctx *context.Context) {
return
}
u = &user_model.User{
Name: getUserName(&gothUser),
FullName: gothUser.Name,
Email: gothUser.Email,
IsActive: !setting.OAuth2Client.RegisterEmailConfirm,
LoginType: auth.OAuth2,
LoginSource: authSource.ID,
LoginName: gothUser.UserID,
IsRestricted: setting.Service.DefaultUserIsRestricted,
Name: getUserName(&gothUser),
FullName: gothUser.Name,
Email: gothUser.Email,
IsActive: !setting.OAuth2Client.RegisterEmailConfirm,
LoginType: auth.OAuth2,
LoginSource: authSource.ID,
LoginName: gothUser.UserID,
}
setUserGroupClaims(authSource, u, &gothUser)

Loading…
Cancel
Save