From 35e4953557076f89b84f2f8c78a3ea00e49e4281 Mon Sep 17 00:00:00 2001 From: Trong Huu Nguyen Date: Tue, 28 Nov 2023 10:08:31 +0100 Subject: [PATCH] fix(session/redis): skip setup if otel is not enabled --- pkg/session/store.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/pkg/session/store.go b/pkg/session/store.go index de72bcf..cac6e11 100644 --- a/pkg/session/store.go +++ b/pkg/session/store.go @@ -42,10 +42,12 @@ func NewStore(cfg *config.Config) (Store, error) { log.Infof("Using Redis as session backing store") } - if err := redisotel.InstrumentTracing(redisClient); err != nil { - return nil, fmt.Errorf("failed to instrument Redis Client: %w", err) + if cfg.OpenTelemetry.Enabled { + if err := redisotel.InstrumentTracing(redisClient); err != nil { + return nil, fmt.Errorf("failed to instrument Redis Client: %w", err) + } + log.Infof("Setup telemtry for Redis") } - log.Infof("Setup telemtry for Redis") return NewRedis(redisClient), nil }