diff --git a/cmd/bucket-listobjects-handlers.go b/cmd/bucket-listobjects-handlers.go index 507d51023..c82513b07 100644 --- a/cmd/bucket-listobjects-handlers.go +++ b/cmd/bucket-listobjects-handlers.go @@ -19,7 +19,6 @@ package cmd import ( "context" - "fmt" "net/http" "strconv" "strings" @@ -101,7 +100,6 @@ func (api objectAPIHandlers) ListObjectVersionsHandler(w http.ResponseWriter, r } if err = DecryptETags(ctx, GlobalKMS, listObjectVersionsInfo.Objects); err != nil { - logger.LogIf(ctx, fmt.Errorf("Failed to decrypt ETag: %v", err)) // TODO(aead): Remove once we are confident that decryption does not fail accidentially writeErrorResponse(ctx, w, toAPIError(ctx, err), r.URL) return } @@ -167,7 +165,6 @@ func (api objectAPIHandlers) ListObjectsV2MHandler(w http.ResponseWriter, r *htt } if err = DecryptETags(ctx, GlobalKMS, listObjectsV2Info.Objects); err != nil { - logger.LogIf(ctx, fmt.Errorf("Failed to decrypt ETag: %v", err)) // TODO(aead): Remove once we are confident that decryption does not fail accidentially writeErrorResponse(ctx, w, toAPIError(ctx, err), r.URL) return } @@ -246,7 +243,6 @@ func (api objectAPIHandlers) ListObjectsV2Handler(w http.ResponseWriter, r *http } if err = DecryptETags(ctx, GlobalKMS, listObjectsV2Info.Objects); err != nil { - logger.LogIf(ctx, fmt.Errorf("Failed to decrypt ETag: %v", err)) // TODO(aead): Remove once we are confident that decryption does not fail accidentially writeErrorResponse(ctx, w, toAPIError(ctx, err), r.URL) return } @@ -347,7 +343,6 @@ func (api objectAPIHandlers) ListObjectsV1Handler(w http.ResponseWriter, r *http } if err = DecryptETags(ctx, GlobalKMS, listObjectsInfo.Objects); err != nil { - logger.LogIf(ctx, fmt.Errorf("Failed to decrypt ETag: %v", err)) // TODO(aead): Remove once we are confident that decryption does not fail accidentially writeErrorResponse(ctx, w, toAPIError(ctx, err), r.URL) return } diff --git a/cmd/gateway-main.go b/cmd/gateway-main.go index f4c9a2af9..c8aed3f5f 100644 --- a/cmd/gateway-main.go +++ b/cmd/gateway-main.go @@ -32,7 +32,6 @@ import ( "github.com/gorilla/mux" "github.com/minio/cli" "github.com/minio/madmin-go" - "github.com/minio/minio/internal/color" "github.com/minio/minio/internal/config" xhttp "github.com/minio/minio/internal/http" "github.com/minio/minio/internal/logger" @@ -221,10 +220,6 @@ func StartGateway(ctx *cli.Context, gw Gateway) { // Set when gateway is enabled globalIsGateway = true - // TODO: We need to move this code with globalConfigSys.Init() - // for now keep it here such that "s3" gateway layer initializes - // itself properly when KMS is set. - // Initialize server config. srvCfg := newServerConfig() @@ -384,15 +379,5 @@ func StartGateway(ctx *cli.Context, gw Gateway) { logger.Info("======") } - // TODO: remove the following line by June 1st. - logger.Info( - color.RedBold(` -=================================================================================== -**** WARNING: MinIO Gateway will be removed by June 1st from MinIO repository ***** - -Please read https://github.com/minio/minio/issues/14331 -=================================================================================== -`)) - <-globalOSSignalCh } diff --git a/cmd/iam.go b/cmd/iam.go index 772501dbb..13a5fa0aa 100644 --- a/cmd/iam.go +++ b/cmd/iam.go @@ -1597,11 +1597,6 @@ func (sys *IAMSys) IsAllowedSTS(args iampolicy.Args, parentUser string) bool { return false } if len(policies) == 0 { - // TODO (deprecated in Dec 2021): Only need to handle - // behavior for STS credentials created in older - // releases. Otherwise, reject such cases, once older - // behavior is deprecated. - // If there is no parent policy mapping, we fall back to // using policy claim from JWT. policySet, ok := args.GetPolicies(iamPolicyClaimNameOpenID()) diff --git a/cmd/metacache-server-pool.go b/cmd/metacache-server-pool.go index b4b782ad1..25cb61c8f 100644 --- a/cmd/metacache-server-pool.go +++ b/cmd/metacache-server-pool.go @@ -630,7 +630,6 @@ func (z *erasureServerPools) listMerged(ctx context.Context, o listPathOptions, return err } if allAtEOF { - // TODO" Maybe, maybe not return io.EOF } return nil