Skip to content

daemon: fix silent_for=2562047h overflow log on relay flip (T7)#414

Merged
TeoSlayer merged 1 commit into
mainfrom
fix/t7-relay-flip-silent-overflow
Jul 22, 2026
Merged

daemon: fix silent_for=2562047h overflow log on relay flip (T7)#414
TeoSlayer merged 1 commit into
mainfrom
fix/t7-relay-flip-silent-overflow

Conversation

@TeoSlayer

Copy link
Copy Markdown
Collaborator

writeFrame computed silent_for as time.Since(LastDirectRecv); for a never-direct peer that's time.Since(zero) ≈ 292 years, printing silent_for=2562047h47m16s — 7600× in one daemon log, misleading log-reading agents. Now renders never for the zero case. Found by the pilot-director probe (T7). 🤖 Generated with Claude Code

…peers (T7)

writeFrame logged 'direct path silent, flipping to relay' with
silent_for computed as time.Since(LastDirectRecv(nodeID)); for a peer
that never had a direct receive LastDirectRecv returns the zero Time, so
time.Since(zero) prints ~292 years (MaxInt64 ns) as
'silent_for=2562047h47m16.854775807s'. Seen 7600x in a single daemon
log — misleads log-reading agents/operators. Now: detect the zero case
(neverDirect) and render silent_for='never'; the flip itself is correct
(a relay-only peer legitimately using relay), only the duration display
was garbage. Also truncate to whole seconds for the normal case.

Reported as T7 in the pilot-director probe (2026-07-20).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@TeoSlayer
TeoSlayer merged commit 4b2e863 into main Jul 22, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants