From 80b9a81387032172f1941df08a2ad7b27f7277b9 Mon Sep 17 00:00:00 2001 From: Mojtaba Arezoomand Date: Fri, 2 Jun 2023 15:50:20 -0700 Subject: [PATCH] fix: enhance if statements in recovery middleware --- middlewares/recovery.go | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/middlewares/recovery.go b/middlewares/recovery.go index 798b2b5..331191b 100644 --- a/middlewares/recovery.go +++ b/middlewares/recovery.go @@ -45,13 +45,15 @@ func NewRecoveryWithConfig(cfg RecoveryConfig) kid.MiddlewareFunc { return func(c *kid.Context) { defer func() { if err := recover(); err != nil { - if cfg.LogRecovers && c.Debug() { - fmt.Fprintf(cfg.Writer, "[RECOVERY] panic recovered: %v\n", err) - } + if c.Debug() { + if cfg.LogRecovers { + fmt.Fprintf(cfg.Writer, "[RECOVERY] panic recovered: %v\n", err) + } - if cfg.PrintStacktrace && c.Debug() { - stack := debug.Stack() - fmt.Fprintf(cfg.Writer, "%s", string(stack)) + if cfg.PrintStacktrace { + stack := debug.Stack() + fmt.Fprintf(cfg.Writer, "%s", string(stack)) + } } if cfg.OnRecovery != nil {