Browse Source

Fix password complexity check on registration (#8887)

* Fix registration password complexity

* Fix integration to use a complex password ;)
for-closed-social
guillep2k 5 years ago
committed by Lunny Xiao
parent
commit
6e1912c73a
2 changed files with 7 additions and 2 deletions
  1. +2
    -2
      integrations/signup_test.go
  2. +5
    -0
      routers/user/auth.go

+ 2
- 2
integrations/signup_test.go View File

@ -19,8 +19,8 @@ func TestSignup(t *testing.T) {
req := NewRequestWithValues(t, "POST", "/user/sign_up", map[string]string{ req := NewRequestWithValues(t, "POST", "/user/sign_up", map[string]string{
"user_name": "exampleUser", "user_name": "exampleUser",
"email": "exampleUser@example.com", "email": "exampleUser@example.com",
"password": "examplePassword",
"retype": "examplePassword",
"password": "examplePassword!1",
"retype": "examplePassword!1",
}) })
MakeRequest(t, req, http.StatusFound) MakeRequest(t, req, http.StatusFound)

+ 5
- 0
routers/user/auth.go View File

@ -1070,6 +1070,11 @@ func SignUpPost(ctx *context.Context, cpt *captcha.Captcha, form auth.RegisterFo
ctx.RenderWithErr(ctx.Tr("auth.password_too_short", setting.MinPasswordLength), tplSignUp, &form) ctx.RenderWithErr(ctx.Tr("auth.password_too_short", setting.MinPasswordLength), tplSignUp, &form)
return return
} }
if !password.IsComplexEnough(form.Password) {
ctx.Data["Err_Password"] = true
ctx.RenderWithErr(ctx.Tr("form.password_complexity"), tplSignUp, &form)
return
}
u := &models.User{ u := &models.User{
Name: form.UserName, Name: form.UserName,

Loading…
Cancel
Save