sloglint settings

Author: moukoublen

.golangci.yml

@@ -118,3 +118,12 @@ linters-settings:
     simple: true
     range-loops: true
     for-loops: true
+
+  sloglint:
+    attr-only: true
+    no-global: "all"
+    context: "scope" # report only if a context exists in the scope of the outermost function
+    static-msg: true
+    key-naming-case: snake #  Values: snake, kebab, camel, pascal
+    forbidden-keys:
+      - msg

internal/config/config.go

@@ -22,12 +22,12 @@ func Load(ctx context.Context, envVarPrefix string, defaultConfigs map[string]an
 
 	// Load default values.
 	if err := k.Load(confmap.Provider(defaultConfigs, delim), nil); err != nil {
-		logger.Debug("error during config loading from defaults", logx.Error(err))
+		logger.DebugContext(ctx, "error during config loading from defaults", logx.Error(err))
 	}
 
 	// Load YAML config.
 	if err := k.Load(file.Provider("config.yaml"), yaml.Parser()); err != nil {
-		logger.Debug("error during config loading from yaml file", logx.Error(err))
+		logger.DebugContext(ctx, "error during config loading from yaml file", logx.Error(err))
 		if !errors.Is(err, fs.ErrNotExist) {
 			return k, err
 		}
@@ -39,12 +39,12 @@ func Load(ctx context.Context, envVarPrefix string, defaultConfigs map[string]an
 		"log":  map[string]any{},
 	}
 	if err := k.Load(env.Provider(envVarPrefix, delim, buildEnvVarsNamesMapper(envVarsLevels, envVarPrefix)), nil); err != nil {
-		logger.Warn("error during config loading from env vars", logx.Error(err))
+		logger.WarnContext(ctx, "error during config loading from env vars", logx.Error(err))
 	}
 
 	// Load .env file
 	if err := k.Load(file.Provider(".env"), dotenv.ParserEnv(envVarPrefix, delim, buildEnvVarsNamesMapper(envVarsLevels, envVarPrefix))); err != nil {
-		logger.Warn("error during config loading from dot env file", logx.Error(err))
+		logger.WarnContext(ctx, "error during config loading from dot env file", logx.Error(err))
 		if !errors.Is(err, fs.ErrNotExist) {
 			return k, err
 		}

internal/daemon.go

@@ -37,7 +37,7 @@ func (o *Daemon) OnShutDown(f ...func(context.Context)) {
 }
 
 func (o *Daemon) shutDown(ctx context.Context) {
-	o.logger.Info("starting graceful shutdown")
+	o.logger.InfoContext(ctx, "starting graceful shutdown")
 	deadline := time.Now().Add(o.config.shutdownTimeout)
 	dlCtx, dlCancel := context.WithDeadline(ctx, deadline)
 	o.onShutDownMutex.Lock()

internal/httpx/inbound.go

@@ -48,7 +48,7 @@ func RespondJSON(ctx context.Context, w http.ResponseWriter, statusCode int, bod
 	if body != nil {
 		if err := json.NewEncoder(w).Encode(body); err != nil {
 			logger := logx.GetFromContext(ctx)
-			logger.Error("error during response encoding", logx.Error(err))
+			logger.ErrorContext(ctx, "error during response encoding", logx.Error(err))
 		}
 	}
 }

internal/httpx/router.go

@@ -78,9 +78,9 @@ func LogRoutes(ctx context.Context, r *chi.Mux) {
 	}
 
 	if err := chi.Walk(r, walkFunc); err != nil {
-		logger.Error("error during chi walk", logx.Error(err))
+		logger.ErrorContext(ctx, "error during chi walk", logx.Error(err))
 	} else {
-		logger.Debug("http routes", slog.Any("routes", routes))
+		logger.DebugContext(ctx, "http routes", slog.Any("routes", routes))
 	}
 }