MyFavMirrors/minio
1
0
Fork 0
mirror of https://github.com/minio/minio.git synced 2024-12-24 06:05:55 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix: do not deny admins to change other passwords

Browse Source 
fixes a regression from #11680
This commit is contained in:
Harshavardhana 2021-03-02 17:02:29 -08:00
parent 879599b0cf
commit f96d4cf7d3
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cmd/admin-handlers-users.go
View File

@ -399,7 +399,7 @@ func (a adminAPIHandlers) AddUser(w http.ResponseWriter, r *http.Request) {
AccountName: parentUser, AccountName: parentUser,
Action: iampolicy.CreateUserAdminAction, Action: iampolicy.CreateUserAdminAction,
ConditionValues: getConditionValues(r, "", parentUser, claims), ConditionValues: getConditionValues(r, "", parentUser, claims),
IsOwner: false, IsOwner: owner,
Claims: claims, Claims: claims,
}) { }) {
writeErrorResponseJSON(ctx, w, errorCodes.ToAPIErr(ErrAccessDenied), r.URL) writeErrorResponseJSON(ctx, w, errorCodes.ToAPIErr(ErrAccessDenied), r.URL)
@ -411,7 +411,7 @@ func (a adminAPIHandlers) AddUser(w http.ResponseWriter, r *http.Request) {
AccountName: accessKey, AccountName: accessKey,
Action: iampolicy.CreateUserAdminAction, Action: iampolicy.CreateUserAdminAction,
ConditionValues: getConditionValues(r, "", accessKey, claims), ConditionValues: getConditionValues(r, "", accessKey, claims),
IsOwner: false, IsOwner: owner,
Claims: claims, Claims: claims,
DenyOnly: true, // check if changing password is explicitly denied. DenyOnly: true, // check if changing password is explicitly denied.
}) { }) {