Fix allot of things

Author: MonkyMars

gecho.go

@@ -47,8 +47,21 @@ var ParseLogLevel = utils.ParseLevel
 var Field = utils.Field
 var WithCallerSkip = utils.WithCallerSkip
 
-// Logger type
+// Logger config functions
+var NewConfig = utils.NewConfig
+var WithLogLevel = utils.WithLogLevel
+var WithLogFormat = utils.WithLogFormat
+var WithColorize = utils.WithColorize
+var WithShowCaller = utils.WithShowCaller
+var WithTimeFormat = utils.WithTimeFormat
+var WithOutput = utils.WithOutput
+var WithErrorOutput = utils.WithErrorOutput
+var WithDefaultCallerSkip = utils.WithDefaultCallerSkip
+
+// Logger types
 type Logger = utils.Logger
+type LoggerConfig = utils.Config
+type LoggerOptions = utils.LoggerOptions
 
 // Log levels
 var (

handlers/handlers.go

@@ -55,7 +55,6 @@ func (h *Handlers) HandleLogging(next http.Handler, logger *utils.Logger) http.H
 			)
 		} else {
 			logger.Info(
-				"HTTP request completed",
 				utils.Field("method", r.Method),
 				utils.Field("path", r.URL.Path),
 				utils.Field("status", wrapper.statusCode),

handlers/handlers_test.go

@@ -384,7 +384,8 @@ func TestHandleLogging(t *testing.T) {
 		})
 
 		handlers := NewHandlers()
-		loggingHandler := handlers.HandleLogging(testHandler, utils.NewDefaultLogger())
+		logger := utils.NewLogger(utils.NewConfig(utils.WithShowCaller(false)))
+		loggingHandler := handlers.HandleLogging(testHandler, logger)
 
 		w := httptest.NewRecorder()
 		r := httptest.NewRequest(http.MethodGet, "/api/data", nil)

utils/logger.go

@@ -93,6 +93,64 @@ func DefaultConfig() Config {
 	}
 }
 
+type LoggerOptions func(*Config)
+
+func NewConfig(options ...LoggerOptions) Config {
+	config := DefaultConfig()
+	for _, option := range options {
+		option(&config)
+	}
+	return config
+}
+
+func WithLogLevel(level Level) LoggerOptions {
+	return func(c *Config) {
+		c.Level = level
+	}
+}
+
+func WithLogFormat(format Format) LoggerOptions {
+	return func(c *Config) {
+		c.Format = format
+	}
+}
+
+func WithColorize(colorize bool) LoggerOptions {
+	return func(c *Config) {
+		c.Colorize = colorize
+	}
+}
+
+func WithShowCaller(showCaller bool) LoggerOptions {
+	return func(c *Config) {
+		c.ShowCaller = showCaller
+	}
+}
+
+func WithTimeFormat(timeFormat string) LoggerOptions {
+	return func(c *Config) {
+		c.TimeFormat = timeFormat
+	}
+}
+
+func WithOutput(output io.Writer) LoggerOptions {
+	return func(c *Config) {
+		c.Output = output
+	}
+}
+
+func WithErrorOutput(errorOutput io.Writer) LoggerOptions {
+	return func(c *Config) {
+		c.ErrorOutput = errorOutput
+	}
+}
+
+func WithDefaultCallerSkip(callerSkip int) LoggerOptions {
+	return func(c *Config) {
+		c.CallerSkip = callerSkip
+	}
+}
+
 // Logger is a structured, thread-safe logger
 type Logger struct {
 	mu     sync.Mutex
@@ -123,13 +181,8 @@ func (l *Logger) WithFields(fields map[string]any) *Logger {
 	l.mu.Lock()
 	defer l.mu.Unlock()
 
-	newFields := make(map[string]any)
-	for k, v := range l.fields {
-		newFields[k] = v
-	}
-	for k, v := range fields {
-		newFields[k] = v
-	}
+	newFields := maps.Clone(l.fields)
+	maps.Copy(newFields, fields)
 
 	return &Logger{
 		config: l.config,