From 559d07562785e98125356842eec6d3c05c9666c5 Mon Sep 17 00:00:00 2001 From: Harshavardhana Date: Tue, 13 Jul 2021 09:32:35 -0700 Subject: [PATCH] fix: properly send group unset policy (#12694) In distributed setups `mc admin group unset` was not correctly working due to incorrect handling of query parameters between peers. --- cmd/peer-rest-server.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/cmd/peer-rest-server.go b/cmd/peer-rest-server.go index 0903374e8..f75a4ad39 100644 --- a/cmd/peer-rest-server.go +++ b/cmd/peer-rest-server.go @@ -129,8 +129,8 @@ func (s *peerRESTServer) LoadPolicyMappingHandler(w http.ResponseWriter, r *http s.writeErrorResponse(w, errors.New("user-or-group is missing")) return } - _, isGroup := vars[peerRESTIsGroup] + _, isGroup := r.URL.Query()[peerRESTIsGroup] if err := globalIAMSys.LoadPolicyMapping(objAPI, userOrGroup, isGroup); err != nil { s.writeErrorResponse(w, err) return @@ -384,7 +384,6 @@ func (s *peerRESTServer) NetInfoHandler(w http.ResponseWriter, r *http.Request) } if n != r.ContentLength { err := fmt.Errorf("Subnet health: short read: expected %d found %d", r.ContentLength, n) - logger.LogIf(ctx, err) w.Header().Set("FinalStatus", err.Error()) return