fix: remove duplicate time log field (#180)

thoas · Oct 9, 2023 · 7a7e2ad · 7a7e2ad
1 parent 69ec446
commit 7a7e2ad
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/middleware/log.go b/middleware/log.go
@@ -7,12 +7,11 @@ import (
 
 	"github.com/gin-gonic/gin"
 	"github.com/google/uuid"
-	"github.com/thoas/picfit/config"
 	"github.com/thoas/picfit/constants"
 	loggerpkg "github.com/thoas/picfit/logger"
 )
 
-func NewLogger(cfg *config.Config, logger *slog.Logger) gin.HandlerFunc {
+func NewLogger(logger *slog.Logger) gin.HandlerFunc {
 	return func(c *gin.Context) {
 		var (
 			ctx       = c.Request.Context()
@@ -27,7 +26,6 @@ func NewLogger(cfg *config.Config, logger *slog.Logger) gin.HandlerFunc {
 
 		c.Next()
 
-		end := time.Now()
 		attributes := []slog.Attr{
 			slog.Int("status", c.Writer.Status()),
 			slog.String("method", c.Request.Method),
@@ -36,7 +34,6 @@ func NewLogger(cfg *config.Config, logger *slog.Logger) gin.HandlerFunc {
 			slog.String("ip", c.ClientIP()),
 			slog.Duration("duration", time.Since(start)),
 			slog.String("user-agent", c.Request.UserAgent()),
-			slog.Time("time", end.UTC()),
 		}
 
 		if len(c.Errors) > 0 {

diff --git a/server/http.go b/server/http.go
@@ -76,7 +76,7 @@ func (s *HTTPServer) Init() error {
 		router.Use(middleware.Recover)
 	}
 
-	router.Use(middleware.NewLogger(s.config, s.processor.Logger))
+	router.Use(middleware.NewLogger(s.processor.Logger))
 	router.Use(middleware.Metrics)
 
 	if s.config.Sentry != nil {