From 580c2b4ad4dd14b400927f8ebded7e6d0d6e56a2 Mon Sep 17 00:00:00 2001 From: Chris Lu Date: Wed, 28 Jan 2026 11:27:18 -0800 Subject: [PATCH] command: fix stale error variable logging in filer serving goroutines - Use local 'err' variable instead of stale 'e' from outer scope - Applied to both TLS and non-TLS paths for local listener --- weed/command/filer.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/weed/command/filer.go b/weed/command/filer.go index d268c98da..8db4f8c85 100644 --- a/weed/command/filer.go +++ b/weed/command/filer.go @@ -477,7 +477,7 @@ func (fo *FilerOptions) startFiler() { if filerLocalListener != nil { go func() { if err := newHttpServer(defaultMux, tlsConfig).ServeTLS(filerLocalListener, "", ""); err != nil { - glog.Errorf("Filer Fail to serve: %v", e) + glog.Errorf("Filer Fail to serve: %v", err) } }() } @@ -496,7 +496,7 @@ func (fo *FilerOptions) startFiler() { if filerLocalListener != nil { go func() { if err := newHttpServer(defaultMux, nil).Serve(filerLocalListener); err != nil { - glog.Errorf("Filer Fail to serve: %v", e) + glog.Errorf("Filer Fail to serve: %v", err) } }() }