dubbo-go icon indicating copy to clipboard operation
dubbo-go copied to clipboard

日志初始化的时候,希望能规范stdout与stderr,,同时也防止log重复初始化

Open Shinku-Chen opened this issue 2 years ago • 4 comments

dubbo.apache.org/dubbo-go/[email protected]/common/logger 中 初始化config.ZapConfig函数希望能够将 OutputPaths: []string{"stderr"}, 修改为 OutputPaths: []string{"stdout"},

因为每次初始化的时候,都会重新初始化log,导致监控报警这部分出现误报 同时也希望在初始化日志的时候,首先判断一下log有无初始化,这样能够防止log重复初始化

Shinku-Chen avatar Dec 02 '22 09:12 Shinku-Chen

good suggestion.

AlexStocks avatar Dec 02 '22 10:12 AlexStocks

https://github.com/dubbogo/dubbo-go-boot @cjphaha 可以看下这个里面的代码整理下

zhaoyunxing92 avatar Dec 02 '22 14:12 zhaoyunxing92

assign https://github.com/codingadai

AlexStocks avatar Jan 13 '23 13:01 AlexStocks

OutputPaths的默认值应该是源自zap,改为stdout,是否要考虑先后兼容性?这个在zap有个issue讨论过。 @zhaoyunxing92

logger初始化的地方有3处:

  • github.com/dubbogo/[email protected]/log/logger/logger.go:29 (init函数里,这块会最先执行)
  • dubbo-go/config/root_config.go:127 (根据配置文件初始化logger)
  • dubbo-go/config/root_config.go:134 (如果有config-center配置,会从config-center中重新加载配置再初始化一次) 最主要是gost logger init函数和 dubbo-go中的 这次重复,我看下怎么改进更合适。

CodingAdai avatar Feb 07 '23 07:02 CodingAdai