Logger周り見直し

[H1rono]

#825 で苦しさを感じたので

• zapは引き続き使いたい
• router.HandlersにLoggerフィールドがあるが、これだとecho.Context外なのでservice/webhook.goからアクセスできない
• model層とent層も見たい
  • model層ではそもそもLoggerにアクセスできない。かと言って雑にctxに突っ込むのは違う
    slog時代のGoではloggerをcontextで引きまわさなくて良い気がする - Diary of a Perpetual Student
  • ent層ではlog.Printlnが使われている。
    

    Jomon/ent/client.go

    Line 103 in a637f70

    cfg := config{log: log.Println, hooks: &hooks{}, inters: &inters{}}

    
    Loggerは設定可能っぽいのでこれを使う
    

    Jomon/ent/client.go

    Lines 126 to 130 in a637f70

    	func Log(fn func(...any)) Option {
    	return func(c *config) {
    	c.log = fn
    	}
    	}

[H1rono]

とは言ってもzap.Withでフィールドを追加したい
つまりdefault logger per goroutineが欲しいんだけど、これはcontextに突っ込むのが手っ取り早いか...

• なぜ Go ではロガーをコンストラクタ DI してはならないのか

[H1rono]

loggingパッケージ復活かなあ
#807 で飛ばしたものとは全く違うもので, context.WithValueとcontext.Valueを使ったloggerの引き回しを管理する役割

[H1rono]

echoとentのlogger設定はzap.NewStdLogAtあたりから繋げられそう

[H1rono]

echo.New見るにむやみに触らん方がいいな...

[H1rono]

https://echo.labstack.com/docs/customization