🚀 Consistent way of logging and fix middleware log format #2432 (#2444)

- change log patter
gofiber · May 1, 2023 · 3a7dbd0 · 3a7dbd0
1 parent a59d9ba
commit 3a7dbd0
Show file tree

Hide file tree

Showing 8 changed files with 15 additions and 15 deletions.
diff --git a/middleware/cache/config.go b/middleware/cache/config.go
@@ -102,11 +102,11 @@ func configDefault(config ...Config) Config {
 
 	// Set default values
 	if cfg.Store != nil {
-		log.Printf("[CACHE] - [Warning] Store is deprecated, please use Storage\n")
+		log.Printf("[Warning] - [CACHE] Store is deprecated, please use Storage\n")
 		cfg.Storage = cfg.Store
 	}
 	if cfg.Key != nil {
-		log.Printf("[CACHE] - [Warning] Key is deprecated, please use KeyGenerator\n")
+		log.Printf("[Warning] - [CACHE] Key is deprecated, please use KeyGenerator\n")
 		cfg.KeyGenerator = cfg.Key
 	}
 	if cfg.Next == nil {

diff --git a/middleware/cors/cors.go b/middleware/cors/cors.go
@@ -98,7 +98,7 @@ func New(config ...Config) fiber.Handler {
 
 	// Warning logs if both AllowOrigins and AllowOriginsFunc are set
 	if cfg.AllowOrigins != ConfigDefault.AllowOrigins && cfg.AllowOriginsFunc != nil {
-		log.Printf("[CORS] - [Warning] Both 'AllowOrigins' and 'AllowOriginsFunc' have been defined.\n")
+		log.Printf("[Warning] - [CORS] Both 'AllowOrigins' and 'AllowOriginsFunc' have been defined.\n")
 	}
 
 	// Convert string to slice

diff --git a/middleware/csrf/config.go b/middleware/csrf/config.go
@@ -132,15 +132,15 @@ func configDefault(config ...Config) Config {
 
 	// Set default values
 	if cfg.TokenLookup != "" {
-		log.Printf("[CSRF] - [Warning] TokenLookup is deprecated, please use KeyLookup\n")
+		log.Printf("[Warning] - [CSRF] TokenLookup is deprecated, please use KeyLookup\n")
 		cfg.KeyLookup = cfg.TokenLookup
 	}
 	if int(cfg.CookieExpires.Seconds()) > 0 {
-		log.Printf("[CSRF] - [Warning] CookieExpires is deprecated, please use Expiration\n")
+		log.Printf("[Warning] - [CSRF] CookieExpires is deprecated, please use Expiration\n")
 		cfg.Expiration = cfg.CookieExpires
 	}
 	if cfg.Cookie != nil {
-		log.Printf("[CSRF] - [Warning] Cookie is deprecated, please use Cookie* related fields\n")
+		log.Printf("[Warning] - [CSRF] Cookie is deprecated, please use Cookie* related fields\n")
 		if cfg.Cookie.Name != "" {
 			cfg.CookieName = cfg.Cookie.Name
 		}

diff --git a/middleware/idempotency/idempotency.go b/middleware/idempotency/idempotency.go
@@ -92,7 +92,7 @@ func New(config ...Config) fiber.Handler {
 		}
 		defer func() {
 			if err := cfg.Lock.Unlock(key); err != nil {
-				log.Printf("[IDEMPOTENCY] - [Error] failed to unlock key %q: %v", key, err)
+				log.Printf("[Error] - [IDEMPOTENCY] failed to unlock key %q: %v", key, err)
 			}
 		}()
 

diff --git a/middleware/limiter/config.go b/middleware/limiter/config.go
@@ -95,15 +95,15 @@ func configDefault(config ...Config) Config {
 
 	// Set default values
 	if int(cfg.Duration.Seconds()) > 0 {
-		log.Printf("[LIMITER] - [Warning] Duration is deprecated, please use Expiration\n")
+		log.Printf("[Warning] - [LIMITER] Duration is deprecated, please use Expiration\n")
 		cfg.Expiration = cfg.Duration
 	}
 	if cfg.Key != nil {
-		log.Printf("[LIMITER] - [Warning] Key is deprecated, please us KeyGenerator\n")
+		log.Printf("[Warning] - [LIMITER] Key is deprecated, please us KeyGenerator\n")
 		cfg.KeyGenerator = cfg.Key
 	}
 	if cfg.Store != nil {
-		log.Printf("[LIMITER] - [Warning] Store is deprecated, please use Storage\n")
+		log.Printf("[Warning] - [LIMITER] Store is deprecated, please use Storage\n")
 		cfg.Storage = cfg.Store
 	}
 	if cfg.Next == nil {

diff --git a/middleware/proxy/proxy.go b/middleware/proxy/proxy.go
@@ -17,7 +17,7 @@ import (
 
 // New is deprecated
 func New(config Config) fiber.Handler {
-	log.Printf("[PROXY] - [Warning] proxy.New is deprecated, please use proxy.Balancer instead\n")
+	log.Printf("[Warning] - [PROXY] proxy.New is deprecated, please use proxy.Balancer instead\n")
 	return Balancer(config)
 }
 

diff --git a/middleware/session/config.go b/middleware/session/config.go
@@ -97,7 +97,7 @@ func configDefault(config ...Config) Config {
 		cfg.Expiration = ConfigDefault.Expiration
 	}
 	if cfg.CookieName != "" {
-		log.Printf("[SESSION] - [Warning] CookieName is deprecated, please use KeyLookup\n")
+		log.Printf("[Warning] - [SESSION] CookieName is deprecated, please use KeyLookup\n")
 		cfg.KeyLookup = fmt.Sprintf("cookie:%s", cfg.CookieName)
 	}
 	if cfg.KeyLookup == "" {

diff --git a/middleware/timeout/timeout.go b/middleware/timeout/timeout.go
@@ -18,7 +18,7 @@ var once sync.Once
 // Find documentation and sample usage on https://docs.gofiber.io/api/middleware/timeout
 func New(handler fiber.Handler, timeout time.Duration) fiber.Handler {
 	once.Do(func() {
-		log.Printf("[TIMEOUT] - [Warning] timeout contains data race issues, not ready for production!")
+		log.Printf("[Warning] - [TIMEOUT] timeout contains data race issues, not ready for production!")
 	})
 
 	if timeout <= 0 {
@@ -32,11 +32,11 @@ func New(handler fiber.Handler, timeout time.Duration) fiber.Handler {
 		go func() {
 			defer func() {
 				if err := recover(); err != nil {
-					log.Printf("[TIMEOUT] - [Warning] recover error %v", err)
+					log.Printf("[Warning] - [TIMEOUT] recover error %v", err)
 				}
 			}()
 			if err := handler(ctx); err != nil {
-				log.Printf("[TIMEOUT] - [Warning] handler error %v", err)
+				log.Printf("[Warning] - [TIMEOUT] handler error %v", err)
 			}
 			ch <- struct{}{}
 		}()