v0.0.472 changed gin engine initialization

ginext/engine.go

@@ -51,6 +51,20 @@ type Options struct {
 
 // NewEngine creates a new (wrapped) ginEngine
 func NewEngine(opt Options) *GinWrapper {
+	ginDebug := langext.Coalesce(opt.GinDebug, true)
+	if ginDebug {
+		gin.SetMode(gin.DebugMode)
+
+		// do not debug-print routes
+		gin.DebugPrintRouteFunc = func(_, _, _ string, _ int) {}
+
+	} else {
+		gin.SetMode(gin.ReleaseMode)
+
+		// do not debug-print routes
+		gin.DebugPrintRouteFunc = func(_, _, _ string, _ int) {}
+	}
+
 	engine := gin.New()
 
 	wrapper := &GinWrapper{
@@ -58,7 +72,7 @@ func NewEngine(opt Options) *GinWrapper {
 		opt:                   opt,
 		suppressGinLogs:       langext.Coalesce(opt.SuppressGinLogs, false),
 		allowCors:             langext.Coalesce(opt.AllowCors, false),
-		ginDebug:              langext.Coalesce(opt.GinDebug, true),
+		ginDebug:              ginDebug,
 		bufferBody:            langext.Coalesce(opt.BufferBody, false),
 		requestTimeout:        langext.Coalesce(opt.Timeout, 24*time.Hour),
 		listenerBeforeRequest: opt.ListenerBeforeRequest,
@@ -72,24 +86,10 @@ func NewEngine(opt Options) *GinWrapper {
 		engine.Use(CorsMiddleware())
 	}
 
-	if wrapper.ginDebug {
-		gin.SetMode(gin.DebugMode)
-
-		// do not debug-print routes
-		gin.DebugPrintRouteFunc = func(_, _, _ string, _ int) {}
-
-		if !wrapper.suppressGinLogs {
-			ginlogger := gin.Logger()
-			engine.Use(func(context *gin.Context) { ginlogger(context) })
-		}
-
-	} else {
-		gin.SetMode(gin.ReleaseMode)
-
-		// do not debug-print routes
-		gin.DebugPrintRouteFunc = func(_, _, _ string, _ int) {}
+	if ginDebug && !wrapper.suppressGinLogs {
+		ginlogger := gin.Logger()
+		engine.Use(func(context *gin.Context) { ginlogger(context) })
 	}
-
 	return wrapper
 }