improve logging

optimizely · Nov 23, 2023 · 12773d1 · 12773d1
1 parent 2204260
commit 12773d1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/cmd/optimizely/main.go b/cmd/optimizely/main.go
@@ -33,6 +33,7 @@ import (
 	"github.com/spf13/viper"
 
 	"github.com/optimizely/agent/config"
+	"github.com/optimizely/agent/pkg/handlers"
 	"github.com/optimizely/agent/pkg/metrics"
 	"github.com/optimizely/agent/pkg/optimizely"
 	"github.com/optimizely/agent/pkg/routers"
@@ -267,6 +268,8 @@ func main() {
 
 	ctx, cancel := context.WithCancel(context.Background()) // Create default service context
 	defer cancel()
+	ctx = context.WithValue(ctx, handlers.LoggerKey, &log.Logger)
+
 	sg := server.NewGroup(ctx, conf.Server) // Create a new server group to manage the individual http listeners
 	optlyCache := optimizely.NewCache(ctx, *conf, sdkMetricsRegistry)
 	optlyCache.Init(conf.SDKKeys)

diff --git a/pkg/handlers/webhook.go b/pkg/handlers/webhook.go
@@ -164,7 +164,7 @@ func (h *OptlyWebhookHandler) HandleWebhook(w http.ResponseWriter, r *http.Reque
 func (h *OptlyWebhookHandler) StartSyncer(ctx context.Context) error {
 	logger, ok := ctx.Value(LoggerKey).(*zerolog.Logger)
 	if !ok {
-		logger = &zerolog.Logger{}
+		logger = &log.Logger
 	}
 
 	if !h.syncEnabled {
@@ -185,8 +185,10 @@ func (h *OptlyWebhookHandler) StartSyncer(ctx context.Context) error {
 				return
 			case key := <-dataCh:
 				h.optlyCache.UpdateConfigs(key)
+				logger.Debug().Msg("datafile synced successfully")
 			}
 		}
 	}()
+	logger.Debug().Msg("datafile syncer is started")
 	return nil
 }