fix: sentry request logging

chore: attach release version info to sentry events

config/config.go

@@ -470,13 +470,14 @@ func Get() *Config {
 
 func Load(configFlag string, version string) *Config {
 	config := &Config{}
-
 	if err := configor.New(&configor.Config{}).Load(config, configFlag); err != nil {
 		Log().Fatal("failed to read config", "error", err)
 	}
 
 	env = config.Env
 
+	InitLogger(config.IsDev())
+
 	config.Version = strings.TrimSpace(version)
 	tagVersionMatch, _ := regexp.MatchString(`\d+\.\d+\.\d+`, version)
 	if tagVersionMatch {
@@ -514,7 +515,7 @@ func Load(configFlag string, version string) *Config {
 			config.Sentry.Environment = config.Env
 		}
 		slog.Info("enabling sentry integration", "environment", config.Sentry.Environment)
-		initSentry(config.Sentry, config.IsDev())
+		initSentry(config.Sentry, config.IsDev(), config.Version)
 	}
 
 	if config.App.DataRetentionMonths <= 0 {

config/logging.go

@@ -0,0 +1,16 @@
+package config
+
+import (
+	"log/slog"
+	"os"
+)
+
+func InitLogger(isDev bool) {
+	var handler slog.Handler
+	if isDev {
+		handler = slog.NewTextHandler(os.Stdout, &slog.HandlerOptions{Level: slog.LevelDebug})
+	} else {
+		handler = slog.NewJSONHandler(os.Stdout, &slog.HandlerOptions{Level: slog.LevelInfo})
+	}
+	slog.SetDefault(slog.New(handler))
+}

config/sentry.go

@@ -45,7 +45,11 @@ func (l *SentryLogger) Fatal(msg string, args ...any) {
 }
 
 func (l *SentryLogger) Request(r *http.Request) *slog.Logger {
-	return l.Logger.With(slog.Any("http_request", r))
+	l.Logger = l.Logger.With("request", r)
+	if uid := getPrincipal(r); uid != "" {
+		l.Logger = l.Logger.With(slog.Group("user", slog.String("id", uid)))
+	}
+	return l.Logger
 }
 
 var excludedRoutes = []string{
@@ -55,11 +59,12 @@ var excludedRoutes = []string{
 	"GET /docs",
 }
 
-func initSentry(config sentryConfig, debug bool) {
+func initSentry(config sentryConfig, debug bool, releaseVersion string) {
 	if err := sentry.Init(sentry.ClientOptions{
 		Dsn:              config.Dsn,
 		Debug:            debug,
 		Environment:      config.Environment,
+		Release:          releaseVersion,
 		AttachStacktrace: true,
 		EnableTracing:    config.EnableTracing,
 		TracesSampler: func(ctx sentry.SamplingContext) float64 {
@@ -75,12 +80,7 @@ func initSentry(config sentryConfig, debug bool) {
 			return float64(config.SampleRate)
 		},
 		BeforeSend: func(event *sentry.Event, hint *sentry.EventHint) *sentry.Event {
-			r, ok := event.Contexts["extra"]["http_request"]
-			if ok {
-				if uid := getPrincipal(r.(*http.Request)); uid != "" {
-					event.User.ID = uid
-				}
-			}
+			// optional pre-processing before sending the event off
 			return event
 		},
 	}); err != nil {