NET-1227: fix update password on update user handler (#3077)

* 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
This commit is contained in:
Abhishek K 2024-08-27 15:24:18 +05:30 committed by GitHub
parent d532060c50
commit 0a1558d5ac
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -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 {
@ -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
}