From 0a58aa146a14f82af2bdaa5dd7d7062d71040f6b Mon Sep 17 00:00:00 2001 From: Kpa-clawbot Date: Thu, 28 May 2026 23:27:58 -0700 Subject: [PATCH] fix(ingestor): silence per-message naive-timestamp log (#1478 followup) (#1479) Operator on prod reports the per-message naive-timestamp warning drowns the log when an observer's local clock isn't UTC. Since observer.last_seen already uses ingest time regardless of envelope (#1466), and per-packet rxTime is already clamped (#1464), the per-message console log adds nothing actionable. This PR silences the log. #1478 tracks the proper followup: surface broken observers in the UI (chip + banner on observer detail). Backend-only, hot-deployable via image pull (no API/schema change). Co-authored-by: openclaw-bot --- cmd/ingestor/main.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/cmd/ingestor/main.go b/cmd/ingestor/main.go index 878849a8..c8b2154c 100644 --- a/cmd/ingestor/main.go +++ b/cmd/ingestor/main.go @@ -1147,7 +1147,10 @@ func resolveRxTime(msg map[string]interface{}, tag string) string { delta = -delta } if delta > naiveTolerance { - log.Printf("MQTT [%s] naive timestamp %q off by %s, using ingest time", tag, raw, delta.Round(time.Second)) + // #1478: log silenced — was firing per-message + drowning logs. + // observer.last_seen uses ingest time regardless (#1466), and + // the bad observer should be surfaced in the UI (see #1478) + // rather than per-message console noise. return now.Format(time.RFC3339) } }