From 0f591d245d3a1810b0e6aebbf3f32937227f68f8 Mon Sep 17 00:00:00 2001 From: Shireesh Anjal <355479+anjalshireesh@users.noreply.github.com> Date: Fri, 20 Jan 2023 07:35:44 +0530 Subject: [PATCH] fix: incorrect anonymization of drive endpoint (#16442) --- cmd/admin-handlers.go | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/cmd/admin-handlers.go b/cmd/admin-handlers.go index 25e8d43b0..7cc9db754 100644 --- a/cmd/admin-handlers.go +++ b/cmd/admin-handlers.go @@ -48,6 +48,7 @@ import ( "github.com/klauspost/compress/zip" "github.com/minio/madmin-go/v2" "github.com/minio/madmin-go/v2/estream" + "github.com/minio/minio-go/v7/pkg/set" "github.com/minio/minio/internal/dsync" "github.com/minio/minio/internal/handlers" xhttp "github.com/minio/minio/internal/http" @@ -2859,10 +2860,16 @@ func createHostAnonymizer() map[string]string { } hostAnonymizer := map[string]string{} + hosts := set.NewStringSet() + srvrIdx := 0 for poolIdx, pool := range globalEndpoints { - for srvrIdx, endpoint := range pool.Endpoints { - anonymizeHost(hostAnonymizer, endpoint, poolIdx+1, srvrIdx+1) + for _, endpoint := range pool.Endpoints { + if !hosts.Contains(endpoint.Host) { + hosts.Add(endpoint.Host) + srvrIdx++ + } + anonymizeHost(hostAnonymizer, endpoint, poolIdx+1, srvrIdx) } } return hostAnonymizer