bugfix dubbo-go issue: nacos-go-sdk stdout redirect #1960 (#489)

fix:490
Co-authored-by: chenzf72 <[email protected]>

Author: chenzfp

Diff for: clients/config_client/config_client.go

@@ -101,6 +101,7 @@ func NewConfigClient(nc nacos_client.INacosClient) (*ConfigClient, error) {
 		LogRollingConfig: clientConfig.LogRollingConfig,
 		LogDir:           clientConfig.LogDir,
 		CustomLogger:     clientConfig.CustomLogger,
+		LogStdout:        clientConfig.LogStdout,
 	}
 	err = logger.InitLogger(loggerConfig)
 	if err != nil {

Diff for: clients/naming_client/naming_client.go

@@ -74,6 +74,7 @@ func NewNamingClient(nc nacos_client.INacosClient) (NamingClient, error) {
 		LogRollingConfig: clientConfig.LogRollingConfig,
 		LogDir:           clientConfig.LogDir,
 		CustomLogger:     clientConfig.CustomLogger,
+		LogStdout:        clientConfig.LogStdout,
 	}
 	err = logger.InitLogger(loggerConfig)
 	if err != nil {

Diff for: common/constant/client_config_options.go

@@ -180,3 +180,10 @@ func WithLogRollingConfig(rollingConfig *lumberjack.Logger) ClientOption {
 		config.LogRollingConfig = rollingConfig
 	}
 }
+
+// WithLogStdout ...
+func WithLogStdout(logStdout bool) ClientOption {
+	return func(config *ClientConfig) {
+		config.LogStdout = logStdout
+	}
+}

Diff for: common/constant/config.go

@@ -52,4 +52,5 @@ type ClientConfig struct {
 	ContextPath          string                 // the nacos server contextpath
 	LogRollingConfig     *lumberjack.Logger     // the log rolling config
 	CustomLogger         logger.Logger          // the custom log interface ,With a custom Logger (nacos sdk will not provide log cutting and archiving capabilities)
+	LogStdout            bool                   // the stdout redirect for log, default is false
 }

Diff for: common/logger/logger.go

@@ -45,6 +45,7 @@ type Config struct {
 	LogRollingConfig *lumberjack.Logger
 	LogDir           string
 	CustomLogger     Logger
+	LogStdout        bool
 }
 
 type SamplingConfig struct {
@@ -108,7 +109,7 @@ func initNacosLogger(config Config) (Logger, error) {
 	encoder := getEncoder()
 	writer := config.getLogWriter()
 
-	core := zapcore.NewCore(zapcore.NewConsoleEncoder(encoder), zapcore.NewMultiWriteSyncer(writer, zapcore.AddSync(os.Stdout)), logLevel)
+	core := zapcore.NewCore(zapcore.NewConsoleEncoder(encoder), writer, logLevel)
 
 	if config.Sampling != nil {
 		core = zapcore.NewSamplerWithOptions(core, config.Sampling.Tick, config.Sampling.Initial, config.Sampling.Thereafter)
@@ -159,5 +160,8 @@ func (c *Config) getLogWriter() zapcore.WriteSyncer {
 		c.LogRollingConfig = &lumberjack.Logger{}
 	}
 	c.LogRollingConfig.Filename = c.LogDir + string(os.PathSeparator) + c.LogFileName
+	if c.LogStdout {
+		return zapcore.NewMultiWriteSyncer(zapcore.AddSync(c.LogRollingConfig), zapcore.AddSync(os.Stdout))
+	}
 	return zapcore.AddSync(c.LogRollingConfig)
 }

Diff for: example/config/main.go

@@ -48,6 +48,7 @@ func main() {
 		LogDir:              "/tmp/nacos/log",
 		CacheDir:            "/tmp/nacos/cache",
 		LogLevel:            "debug",
+		LogStdout:           true,
 	}
 	//or a more graceful way to create ClientConfig
 	_ = *constant.NewClientConfig(
@@ -57,6 +58,7 @@ func main() {
 		constant.WithLogDir("/tmp/nacos/log"),
 		constant.WithCacheDir("/tmp/nacos/cache"),
 		constant.WithLogLevel("debug"),
+		constant.WithLogStdout(true),
 	)
 
 	// a more graceful way to create config client

Diff for: example/service/main.go

@@ -48,6 +48,7 @@ func main() {
 		CacheDir:            "/tmp/nacos/cache",
 		LogRollingConfig:    &lumberjack.Logger{MaxSize: 10},
 		LogLevel:            "debug",
+		LogStdout:           true,
 	}
 	//or a more graceful way to create ClientConfig
 	_ = *constant.NewClientConfig(
@@ -58,6 +59,7 @@ func main() {
 		constant.WithCacheDir("/tmp/nacos/cache"),
 		constant.WithLogLevel("debug"),
 		constant.WithLogRollingConfig(&lumberjack.Logger{MaxSize: 10}),
+		constant.WithLogStdout(true),
 	)
 
 	// a more graceful way to create naming client