logger: dotenvダンプの秘密漏洩・panic・データ競合を修正 (#16)

[ieee0824]

概要

logger.go の dotenv ダンプ経路(GETENV_DUMP_MODE=dotenv)に集中していた、秘密漏洩フットガン・panic・データ競合・出力破損をまとめて修正します。

Closes #16

背景

セキュリティ監査(敵対的検証済み)で、以下が検出されていました(いずれも opt-in なデバッグ経路ゲートのため深刻度 low ですが、ライブラリがロギング副作用で秘密を漏らしたりホストをクラッシュさせるべきではありません)。

• 環境変数からtime.Durationを取得する #1 / test追加 #2 デフォルト値(秘密になり得る)を stdout に出力(スライス型も含む)
• duration取得する #3 書き込みエラーで panic し全アクセサに伝播 → ホストプロセス DoS
• add readme #4 グローバル Logger.w が無ロックで SetWriter とデータ競合
• time.ParseDuration使う #5 1 レコードを複数 Write に分割しており並行時に行が交錯(監査で 400 行中 153 行破損を実測)

変更点

• secure by default: ダンプのデフォルト値出力を既定でマスク(KEY=)し、Logger.DumpValues(true) で明示的に opt-in したときのみ実値を出力(環境変数からtime.Durationを取得する #1 / test追加 #2)
• Dump の書き込みエラーを panic からベストエフォートの log.Printf に変更(duration取得する #3)
• Logger.w / dumpValues を sync.Mutex で保護(add readme #4)
• 1 レコードを bytes.Buffer で組み立て単一 Write に集約(time.ParseDuration使う #5)
• logger_test.go を追加:マスク既定 / opt-in / スライス / panic 回避 / 並行(-race・行の非交錯)を検証
• README(en/jp)に dotenv ダンプ機能と DumpValues を追記(従来は未記載)

補足(スコープ外)

• add test #14(os.Getenv を毎回読む): example.go が実行時に GETENV_DUMP_MODE をセットして使う動作に依存するため、init キャッシュ化は破壊的。info かつ実害なし(Go 1.21 では O(1))のため据え置き。

挙動変更(破壊的の可能性)

• GETENV_DUMP_MODE=dotenv の出力が KEY=<デフォルト値> → KEY=(マスク)に変わります。従来どおり値を出したい場合は getenv.Logger.DumpValues(true) を呼んでください。dotenv ダンプは README 未記載のデバッグ機能のため影響は限定的と判断しています。

テスト

• go build ./... ✅
• go vet ./... ✅
• go test ./... ✅
• go test -race ./... ✅

🤖 Generated with Claude Code