Skip to content

Commit

Permalink
NET-1227: fix update password on update user handler (#3077)
Browse files Browse the repository at this point in the history
* add list roles to pro and ce

* if not pro set user role to admin

* validate update user

* add separate validation check for password on update

* remove validate check
  • Loading branch information
abhishek9686 committed Aug 27, 2024
1 parent d532060 commit 0a1558d
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions logic/auth.go
Original file line number Diff line number Diff line change
Expand Up @@ -268,10 +268,6 @@ func UpdateUser(userchange, user *models.User) (*models.User, error) {
if _, err := GetUser(user.UserName); err != nil {
return &models.User{}, err
}
err := ValidateUser(userchange)
if err != nil {
return &models.User{}, err
}

queryUser := user.UserName
if userchange.UserName != "" && user.UserName != userchange.UserName {
Expand Down Expand Up @@ -309,7 +305,7 @@ func UpdateUser(userchange, user *models.User) (*models.User, error) {
}
user.UserGroups = userchange.UserGroups
user.NetworkRoles = userchange.NetworkRoles
err = ValidateUser(user)
err := ValidateUser(user)
if err != nil {
return &models.User{}, err
}
Expand Down

0 comments on commit 0a1558d

Please sign in to comment.