initial

internal/logger/logger.go

@@ -0,0 +1,112 @@
+package logger
+
+import (
+	"io"
+	"log/slog"
+	"os"
+	"time"
+
+	"git.eeqj.de/sneak/webhooker/internal/globals"
+	"go.uber.org/fx"
+)
+
+// nolint:revive // LoggerParams is a standard fx naming convention
+type LoggerParams struct {
+	fx.In
+	Globals *globals.Globals
+}
+
+type Logger struct {
+	logger *slog.Logger
+	params LoggerParams
+}
+
+// nolint:revive // lc parameter is required by fx even if unused
+func New(lc fx.Lifecycle, params LoggerParams) (*Logger, error) {
+	l := new(Logger)
+	l.params = params
+
+	// Determine if we're running in a terminal
+	tty := false
+	if fileInfo, _ := os.Stdout.Stat(); (fileInfo.Mode() & os.ModeCharDevice) != 0 {
+		tty = true
+	}
+
+	var handler slog.Handler
+	opts := &slog.HandlerOptions{
+		Level: slog.LevelInfo,
+		ReplaceAttr: func(_ []string, a slog.Attr) slog.Attr { // nolint:revive // groups unused
+			// Always use UTC for timestamps
+			if a.Key == slog.TimeKey {
+				if t, ok := a.Value.Any().(time.Time); ok {
+					return slog.Time(slog.TimeKey, t.UTC())
+				}
+			}
+			return a
+		},
+	}
+
+	if tty {
+		// Use text handler for terminal output (human-readable)
+		handler = slog.NewTextHandler(os.Stdout, opts)
+	} else {
+		// Use JSON handler for production (machine-readable)
+		handler = slog.NewJSONHandler(os.Stdout, opts)
+	}
+
+	l.logger = slog.New(handler)
+
+	// Set as default logger
+	slog.SetDefault(l.logger)
+
+	return l, nil
+}
+
+func (l *Logger) EnableDebugLogging() {
+	// Recreate logger with debug level
+	tty := false
+	if fileInfo, _ := os.Stdout.Stat(); (fileInfo.Mode() & os.ModeCharDevice) != 0 {
+		tty = true
+	}
+
+	var handler slog.Handler
+	opts := &slog.HandlerOptions{
+		Level: slog.LevelDebug,
+		ReplaceAttr: func(_ []string, a slog.Attr) slog.Attr { // nolint:revive // groups unused
+			// Always use UTC for timestamps
+			if a.Key == slog.TimeKey {
+				if t, ok := a.Value.Any().(time.Time); ok {
+					return slog.Time(slog.TimeKey, t.UTC())
+				}
+			}
+			return a
+		},
+	}
+
+	if tty {
+		handler = slog.NewTextHandler(os.Stdout, opts)
+	} else {
+		handler = slog.NewJSONHandler(os.Stdout, opts)
+	}
+
+	l.logger = slog.New(handler)
+	slog.SetDefault(l.logger)
+	l.logger.Debug("debug logging enabled", "debug", true)
+}
+
+func (l *Logger) Get() *slog.Logger {
+	return l.logger
+}
+
+func (l *Logger) Identify() {
+	l.logger.Info("starting",
+		"appname", l.params.Globals.Appname,
+		"version", l.params.Globals.Version,
+		"buildarch", l.params.Globals.Buildarch,
+	)
+}
+
+// Helper methods to maintain compatibility with existing code
+func (l *Logger) Writer() io.Writer {
+	return os.Stdout
+}

internal/logger/logger_test.go

@@ -0,0 +1,65 @@
+package logger
+
+import (
+	"testing"
+
+	"git.eeqj.de/sneak/webhooker/internal/globals"
+	"go.uber.org/fx/fxtest"
+)
+
+func TestNew(t *testing.T) {
+	// Set up globals
+	globals.Appname = "test-app"
+	globals.Version = "1.0.0"
+	globals.Buildarch = "test-arch"
+
+	lc := fxtest.NewLifecycle(t)
+	g, err := globals.New(lc)
+	if err != nil {
+		t.Fatalf("globals.New() error = %v", err)
+	}
+
+	params := LoggerParams{
+		Globals: g,
+	}
+
+	logger, err := New(lc, params)
+	if err != nil {
+		t.Fatalf("New() error = %v", err)
+	}
+
+	if logger.Get() == nil {
+		t.Error("Get() returned nil logger")
+	}
+
+	// Test that we can log without panic
+	logger.Get().Info("test message", "key", "value")
+}
+
+func TestEnableDebugLogging(t *testing.T) {
+	// Set up globals
+	globals.Appname = "test-app"
+	globals.Version = "1.0.0"
+	globals.Buildarch = "test-arch"
+
+	lc := fxtest.NewLifecycle(t)
+	g, err := globals.New(lc)
+	if err != nil {
+		t.Fatalf("globals.New() error = %v", err)
+	}
+
+	params := LoggerParams{
+		Globals: g,
+	}
+
+	logger, err := New(lc, params)
+	if err != nil {
+		t.Fatalf("New() error = %v", err)
+	}
+
+	// Enable debug logging should not panic
+	logger.EnableDebugLogging()
+
+	// Test debug logging
+	logger.Get().Debug("debug message", "test", true)
+}