refactor(log): require destination in calls to NewLogger (#15262)

server/util.go

@@ -64,7 +64,7 @@ func NewDefaultContext() *Context {
 	return NewContext(
 		viper.New(),
 		cmtcfg.DefaultConfig(),
-		log.NewLogger(),
+		log.NewLogger(os.Stdout), // TODO(mr): update NewDefaultContext to accept log destination.
 	)
 }
 
@@ -155,10 +155,10 @@ func InterceptConfigsPreRunHandler(cmd *cobra.Command, customAppConfigTemplate s
 
 	var logger log.Logger
 	if serverCtx.Viper.GetString(flags.FlagLogFormat) == cmtcfg.LogFormatJSON {
-		zl := zerolog.New(os.Stdout).With().Timestamp().Logger()
+		zl := zerolog.New(cmd.OutOrStdout()).With().Timestamp().Logger()
 		logger = log.NewCustomLogger(zl)
 	} else {
-		logger = log.NewLogger()
+		logger = log.NewLogger(cmd.OutOrStdout())
 	}
 
 	// set filter level or keys for the logger if any