From 3058aa6315730722bc92531684b575694de5c69a Mon Sep 17 00:00:00 2001 From: Ross Smith II Date: Tue, 19 Aug 2025 13:36:10 -0700 Subject: [PATCH] chore(slog): re-enable LOGGER_DISCARD (fixes #10262) (#10267) ### Purpose Re-enables LOGGER_DISCARD. See #10262. ### Documentation No changes needed, as the docs already mention this variable. --- internal/slogutil/sloginit.go | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/internal/slogutil/sloginit.go b/internal/slogutil/sloginit.go index b607e75c0..bfd2a5711 100644 --- a/internal/slogutil/sloginit.go +++ b/internal/slogutil/sloginit.go @@ -7,6 +7,7 @@ package slogutil import ( + "io" "log/slog" "os" "strings" @@ -21,10 +22,20 @@ var ( } slogDef = slog.New(&formattingHandler{ recs: []*lineRecorder{GlobalRecorder, ErrorRecorder}, - out: os.Stdout, + out: logWriter(), }) ) +func logWriter() io.Writer { + if os.Getenv("LOGGER_DISCARD") != "" { + // Hack to completely disable logging, for example when running + // benchmarks. + return io.Discard + } + + return os.Stdout +} + func init() { slog.SetDefault(slogDef)