From 2393a13f86ab9f28f1fa4b692ef1e0a8a7a92164 Mon Sep 17 00:00:00 2001 From: Aditya Manthramurthy Date: Thu, 21 Jul 2022 19:52:23 -0700 Subject: [PATCH] Allow site replication config with multiple IDPs (#15361) Fixes a bug that did not let site replication be configured when multiple IDPs are configured. --- internal/config/identity/openid/openid.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/config/identity/openid/openid.go b/internal/config/identity/openid/openid.go index fd77b0510..fc3447a77 100644 --- a/internal/config/identity/openid/openid.go +++ b/internal/config/identity/openid/openid.go @@ -515,7 +515,7 @@ func (r *Config) GetSettings() madmin.OpenIDSettings { hashedSecret = base64.RawURLEncoding.EncodeToString(bs) } if arn != DummyRoleARN { - if res.Roles != nil { + if res.Roles == nil { res.Roles = make(map[string]madmin.OpenIDProviderSettings) } res.Roles[arn.String()] = madmin.OpenIDProviderSettings{