From 930c9b3bebc68ca7b5381c2820fabc4a9a551e95 Mon Sep 17 00:00:00 2001 From: Will Assis <35489495+gassiss@users.noreply.github.com> Date: Fri, 27 Jun 2025 13:22:16 -0400 Subject: [PATCH] fix: double registration panic in auth fallback (#107330) fix double registration panic in auth fallback --- pkg/storage/unified/sql/service.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/storage/unified/sql/service.go b/pkg/storage/unified/sql/service.go index deaa2b177f6..5309e23f729 100644 --- a/pkg/storage/unified/sql/service.go +++ b/pkg/storage/unified/sql/service.go @@ -297,12 +297,13 @@ func ReadGrpcServerConfig(cfg *setting.Cfg) *grpcutils.AuthenticatorConfig { func NewAuthenticatorWithFallback(cfg *setting.Cfg, reg prometheus.Registerer, tracer trace.Tracer, fallback func(context.Context) (context.Context, error)) func(context.Context) (context.Context, error) { authCfg := ReadGrpcServerConfig(cfg) authenticator := grpcutils.NewAuthenticator(authCfg, tracer) + metrics := newMetrics(reg) return func(ctx context.Context) (context.Context, error) { a := &authenticatorWithFallback{ authenticator: authenticator, fallback: fallback, tracer: tracer, - metrics: newMetrics(reg), + metrics: metrics, } return a.Authenticate(ctx) }