maj: better logging via slog defer

context.go

@@ -124,7 +124,7 @@ func (c *icontext) FetchContent(dest interface{}) ErrorHandled {
 	b := c.PostBody()
 
 	if e := json.Unmarshal(b, &dest); e != nil {
-		c.slog.Error("fetching payload", "error", e)
+		c.slog.Error("fetching payload", slog.Any("error", e))
 
 		return errUnprocessablePayload
 	}
@@ -136,7 +136,7 @@ func (c *icontext) FetchContent(dest interface{}) ErrorHandled {
 // this implemtation use validator to anotate & check struct
 func (c *icontext) Validate(dest interface{}) ErrorHandled {
 	if e := validate.Struct(dest); e != nil {
-		c.slog.Error("validating form or query param", "error", e)
+		c.slog.Error("validating form or query param", slog.Any("error", e))
 
 		var ev validator.ValidationErrors
 
@@ -172,7 +172,7 @@ func (c *icontext) DecodeQP(dest interface{}) ErrorHandled {
 	})
 
 	if e := decoder.Decode(dest, m); e != nil {
-		c.slog.Error("validating query params", "error", e)
+		c.slog.Error("validating query params", slog.Any("error", e))
 
 		return NewUnprocessable(NewErrorFromError(e))
 	}

handler.go

@@ -73,7 +73,9 @@ func handleNotFound(c Context) error {
 	fc := c.GetFastContext()
 
 	c.GetStructuredLogger().Info("[!] 404 reached", slog.Group("request",
-		"ip", GetIPFromRequest(fc), "method", fc.Method(), "uri", fc.RequestURI()))
+		slog.String("ip", GetIPFromRequest(fc)),
+		slog.String("method", string(fc.Method())),
+		slog.String("uri", string(fc.RequestURI()))))
 
 	return c.JSONNotFound(json.RawMessage(`{"status":404,"message":"not found"}`))
 }
@@ -82,7 +84,9 @@ func handleNotAllowed(c Context) error {
 	fc := c.GetFastContext()
 
 	c.GetStructuredLogger().Info("[!] 405 reached", slog.Group("request",
-		"ip", GetIPFromRequest(fc), "method", fc.Method(), "uri", fc.RequestURI()))
+		slog.String("ip", GetIPFromRequest(fc)),
+		slog.String("method", string(fc.Method())),
+		slog.String("uri", string(fc.RequestURI()))))
 
 	return c.JSONMethodNotAllowed(json.RawMessage(`{"status":405,"message":"method not allowed"}`))
 }

response.go

@@ -1,5 +1,7 @@
 package webfmwk
 
+import "log/slog"
+
 type (
 	// Header represent a header in a string key:value form.
 	Header [2]string
@@ -55,7 +57,7 @@ func (c *icontext) setHeaders(headers ...Header) {
 	for _, h := range headers {
 		key, val := h[0], h[1]
 		if key == "" || val == "" {
-			c.slog.Warn("can't set header: empty value", "key", key, "val", val)
+			c.slog.Warn("can't set header: empty value", slog.String("key", key), slog.String("val", val))
 
 			return
 		}

response_json.go

@@ -3,6 +3,7 @@ package webfmwk
 import (
 	"bytes"
 	"fmt"
+	"log/slog"
 	"net/http"
 
 	"github.com/segmentio/encoding/json"
@@ -76,7 +77,7 @@ func (c *icontext) JSONBlob(statusCode int, content []byte) error {
 	)
 
 	if e := run(); e != nil {
-		c.slog.Error("cannot prettying the content", "error", e)
+		c.slog.Error("cannot prettying the content", slog.Any("error", e))
 	} else {
 		content = out.Bytes()
 	}

route.go

@@ -2,6 +2,7 @@ package webfmwk
 
 import (
 	"context"
+	"log/slog"
 
 	"github.com/fasthttp/router"
 	"github.com/segmentio/encoding/json"
@@ -190,7 +191,9 @@ func (s *Server) RouteApplier(rpps ...RoutesPerPrefix) {
 				case ANY:
 					s.ANY(prefix+route.Path, route.Handler)
 				default:
-					s.slog.Warn("Cannot load route [%s](%s)", "route", prefix+route.Path, "verbe", route.Verbe)
+					s.slog.Warn("cannot load route",
+						slog.String("route", prefix+route.Path),
+						slog.String("verbe", route.Verbe))
 				}
 			}
 		}
@@ -215,7 +218,7 @@ func (s *Server) GetRouter() *router.Router {
 	if len(s.meta.docHandlers) > 0 {
 		for i := range s.meta.docHandlers {
 			h := s.meta.docHandlers[i]
-			s.slog.Info("load doc handler", "name", h.Name)
+			s.slog.Info("load doc handler", slog.String("name", h.Name))
 			r.ANY(s.meta.prefix+h.Path, s.CustomHandler(h.H))
 		}
 	}
@@ -230,11 +233,11 @@ func (s *Server) GetRouter() *router.Router {
 	// register socket.io (goplog) handlers
 	switch {
 	case s.meta.socketIOHF:
-		s.slog.Info("loading socket io handler func", "path", s.meta.socketIOPath)
+		s.slog.Info("loading socket io handler func", slog.String("path", s.meta.socketIOPath))
 		r.ANY(s.meta.socketIOPath,
 			fasthttpadaptor.NewFastHTTPHandlerFunc(s.meta.socketIOHandlerFunc))
 	case s.meta.socketIOH:
-		s.slog.Info("loading socket io handler", "path", s.meta.socketIOPath)
+		s.slog.Info("loading socket io handler", slog.String("path", s.meta.socketIOPath))
 		r.ANY(s.meta.socketIOPath,
 			fasthttpadaptor.NewFastHTTPHandler(s.meta.socketIOHandler))
 	}

server.go

@@ -98,22 +98,22 @@ func Shutdown() error {
 // Start expose an server to an HTTP endpoint.
 func (s *Server) Start(addr string) {
 	if s.meta.http2 {
-		s.slog.Warn("https endpoints required with http2, skipping", "address", addr)
+		s.slog.Warn("https endpoints required with http2, skipping", slog.String("address", addr))
 
 		return
 	}
 
 	s.internalHandler()
 	s.launcher.Start(func() {
-		s.slog.Debug("http server: starting", "address", addr)
+		s.slog.Debug("http server: starting", slog.String("address", addr))
 
 		go s.pollPingEndpoint(addr)
 
 		if e := s.internalInit(addr).ListenAndServe(addr); e != nil {
-			s.slog.Error("http server", "address", addr, "error", e)
+			s.slog.Error("http server", slog.String("address", addr), slog.Any("error", e))
 		}
 
-		s.slog.Info("http server: done", "address", addr)
+		s.slog.Info("http server: done", slog.String("address", addr))
 	})
 }
 
@@ -124,13 +124,13 @@ func (s *Server) StartTLS(addr string, cfg tls.IConfig) {
 
 	tlsCfg, err := tls.GetTLSCfg(cfg, s.meta.http2)
 	if err != nil {
-		s.slog.Error("loading tls config", "error", err)
+		s.slog.Error("loading tls config", slog.Any("error", err))
 		os.Exit(2)
 	}
 
 	listner, err := tls.LoadListner(addr, tlsCfg)
 	if err != nil {
-		s.slog.Error("loading tls listener", "error", err)
+		s.slog.Error("loading tls listener", slog.Any("error", err))
 		os.Exit(3)
 	}
 
@@ -144,13 +144,14 @@ func (s *Server) StartTLS(addr string, cfg tls.IConfig) {
 	so2 := sOr2(s.meta.http2)
 
 	s.launcher.Start(func() {
-		s.slog.Debug(fmt.Sprintf("%s server: starting", so2), "address", addr)
-		defer s.slog.Info(fmt.Sprintf("%s server: done", so2), "address", addr)
+		s.slog.Debug(fmt.Sprintf("%s server: starting", so2), slog.String("address", addr))
+		defer s.slog.Info(fmt.Sprintf("%s server: done", so2), slog.String("address", addr))
 
 		go s.pollPingEndpoint(addr, cfg)
 
 		if e := server.Serve(listner); e != nil {
-			s.slog.Error(fmt.Sprintf("%s server", so2), "address", addr, "error", e)
+			s.slog.Error(fmt.Sprintf("%s server", so2),
+				slog.String("address", addr), slog.Any("error", e))
 		}
 	})
 }
@@ -189,7 +190,7 @@ func (s *Server) DumpRoutes() map[string][]string {
 
 	for m, p := range all {
 		for i := range p {
-			s.slog.Info("routes", "name", m, "route", p[i])
+			s.slog.Info("routes", slog.String("name", m), slog.String("route", p[i]))
 		}
 	}
 
@@ -230,7 +231,7 @@ func (s *Server) internalInit(addr string) *fasthttp.Server {
 
 	poolOfServers = append(poolOfServers, worker)
 
-	s.slog.Debug("[+] server ", "address", addr, "total", len(poolOfServers))
+	s.slog.Debug("[+] server ", slog.String("address", addr), slog.Int("total", len(poolOfServers)))
 
 	return worker
 }
@@ -266,14 +267,14 @@ func (s *Server) exitHandler(sig ...os.Signal) {
 
 	defer func() {
 		if e := s.Shutdown(); e != nil {
-			s.slog.Error("cannot stop the server", "error", e)
+			s.slog.Error("cannot stop the server", slog.Any("error", e))
 		}
 	}()
 
 	for s.ctx.Err() == nil {
 		select {
 		case si := <-c:
-			s.slog.Info("captured signal, exiting...", "signal", si)
+			s.slog.Info("captured signal, exiting...", slog.String("signal", si.String()))
 
 			return
 		case <-s.ctx.Done():
@@ -287,29 +288,19 @@ func (s *Server) exitHandler(sig ...os.Signal) {
 //
 
 // GetLogger return the used Log instance.
-func (s *Server) GetStructuredLogger() *slog.Logger {
-	return s.slog
-}
+func (s *Server) GetStructuredLogger() *slog.Logger { return s.slog }
 
 // GetLauncher return a pointer to the internal workerLauncher.
-func (s *Server) GetLauncher() WorkerLauncher {
-	return s.launcher
-}
+func (s *Server) GetLauncher() WorkerLauncher { return s.launcher }
 
 // GetContext return the context.Context used.
-func (s *Server) GetContext() context.Context {
-	return s.ctx
-}
+func (s *Server) GetContext() context.Context { return s.ctx }
 
 // GetContext return the server' context cancel func.
-func (s *Server) GetCancel() context.CancelFunc {
-	return s.cancel
-}
+func (s *Server) GetCancel() context.CancelFunc { return s.cancel }
 
 // IsReady return the channel on which `true` is send once the server is up.
-func (s *Server) IsReady() chan bool {
-	return s.isReady
-}
+func (s *Server) IsReady() chan bool { return s.isReady }
 
 // AddHandlers register the Handler handlers. Handler are executed from the top most.
 func (s *Server) addHandlers(h ...Handler) *Server { //nolint: unparam