From 73c7f95d12aa307cc6db9103f05be84544d95efa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mat=C3=ADas=20Insaurralde?= Date: Thu, 26 Mar 2026 20:29:00 -0300 Subject: [PATCH] spindle: replace fmt.Println with structured logger in processPipeline MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Unmarshal errors in processPipeline were written to stdout via fmt.Println, bypassing the slog-based structured logger. Replace with s.l.Error so the error carries the standard fields (timestamp, component, log level) and appears consistently with the rest of spindle's log output. Signed-off-by: Matías Insaurralde --- spindle/server.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spindle/server.go b/spindle/server.go index f1b76e7a..94a2b101 100644 --- a/spindle/server.go +++ b/spindle/server.go @@ -298,7 +298,7 @@ func (s *Spindle) processPipeline(ctx context.Context, src eventconsumer.Source, tpl := tangled.Pipeline{} err := json.Unmarshal(msg.EventJson, &tpl) if err != nil { - fmt.Println("error unmarshalling", err) + s.l.Error("failed to unmarshal pipeline event", "err", err) return err } -- 2.51.2