llvy.ltd/casdoor
1
0
Fork 0
mirror of https://github.com/casdoor/casdoor.git synced 2025-07-03 04:10:20 +08:00
Code Issues Packages Projects Releases Wiki Activity

Improve password length check.

Browse Source
This commit is contained in:
Yang Luo
2021-05-16 21:52:50 +08:00
parent 798386655f
commit e6862713bb
3 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
controllers/user.go
View File

@ -70,6 +70,11 @@ func (c *ApiController) UpdateUser() {
panic(err)
}
if user.DisplayName == "" {
c.ResponseError("Display name cannot be empty")
return
}
c.Data["json"] = wrapActionResponse(object.UpdateUser(id, &user))
c.ServeJSON()
}
@ -160,12 +165,12 @@ func (c *ApiController) SetPassword() {
}
if strings.Index(newPassword, " ") >= 0 {
c.ResponseError("New password contains blank space.")
c.ResponseError("New password cannot contain blank space.")
return
}
if newPassword == "" {
c.ResponseError("Invalid new password")
if len(newPassword) <= 5 {
c.ResponseError("New password must have at least 6 characters")
return
}

8
object/check.go
View File

@ -30,10 +30,10 @@ func init() {
func CheckUserSignup(organizationName string, username string, password string, displayName string, email string, phone string, affiliation string) string {
organization := getOrganization("admin", organizationName)
if len(username) == 0 {
return "username cannot be blank"
} else if len(password) == 0 {
return "password cannot be blank"
if len(username) <= 2 {
return "username must have at least 3 characters"
} else if len(password) <= 5 {
return "password must have at least 6 characters"
} else if organization == nil {
return "organization does not exist"
} else if reWhiteSpace.MatchString(username) {

2
web/src/UserEditPage.js
View File

@ -173,7 +173,7 @@ class UserEditPage extends React.Component {
{i18next.t("general:Organization")}:
</Col>
<Col span={22} >
<Select virtual={false} style={{width: '100%'}} value={this.state.user.owner} onChange={(value => {this.updateUserField('owner', value);})}>
<Select virtual={false} style={{width: '100%'}} disabled={!Setting.isAdminUser(this.props.account)} value={this.state.user.owner} onChange={(value => {this.updateUserField('owner', value);})}>
{
this.state.organizations.map((organization, index) => <Option key={index} value={organization.name}>{organization.name}</Option>)
}