Fix importIAM issue with importing implied policies (minio#20956)

taran-p · web-flow · commit 39df134204b1 · 2025-02-19T10:10:53.000-08:00
diff --git a/cmd/admin-handlers-users.go b/cmd/admin-handlers-users.go
@@ -2441,7 +2441,7 @@ func (a adminAPIHandlers) importIAM(w http.ResponseWriter, r *http.Request, apiV
 				}
 				var sp *policy.Policy
 				var err error
-				if len(svcAcctReq.SessionPolicy) > 0 {
+				if len(svcAcctReq.SessionPolicy) > 0 && !bytes.Equal(svcAcctReq.SessionPolicy, []byte("null")) {
 					sp, err = policy.ParseConfig(bytes.NewReader(svcAcctReq.SessionPolicy))
 					if err != nil {
 						writeErrorResponseJSON(ctx, w, importError(ctx, err, allSvcAcctsFile, user), r.URL)

Original file line number	Diff line number	Diff line change
`@@ -2441,7 +2441,7 @@ func (a adminAPIHandlers) importIAM(w http.ResponseWriter, r *http.Request, apiV`
`2441`	`2441`	`}`
`2442`	`2442`	`var sp *policy.Policy`
`2443`	`2443`	`var err error`
`2444`		`- if len(svcAcctReq.SessionPolicy) > 0 {`
	`2444`	`+ if len(svcAcctReq.SessionPolicy) > 0 && !bytes.Equal(svcAcctReq.SessionPolicy, []byte("null")) {`
`2445`	`2445`	`sp, err = policy.ParseConfig(bytes.NewReader(svcAcctReq.SessionPolicy))`
`2446`	`2446`	`if err != nil {`
`2447`	`2447`	`writeErrorResponseJSON(ctx, w, importError(ctx, err, allSvcAcctsFile, user), r.URL)`