Skip to content

Log a lost Postgres listener as a warning, and stamp 5.1.0 - #76

Merged
windischb merged 1 commit into
developfrom
claude/signalarr-backplane-postgres-decision-775a4c
Sep 5, 2026
Merged

windischb merged 1 commit into
developfrom
claude/signalarr-backplane-postgres-decision-775a4c

Conversation

@windischb

Copy link
Copy Markdown
Collaborator

Why

Modgud runs 5.1.0-beta.3 in production and reported one thing: the lost-listener log line is an error with a stack trace although the node is back 0.6 s later and the message itself promises a replay. Anyone with error alerting gets paged on every proxy idle-timeout.

What

  • The listener loss is logged as a warning naming the cause (ex.Message), no stack trace. The error with the exception is reserved for reconnects that keep failing: third consecutive attempt, which with the 0.5 s / 1 s / 2 s backoff is about 3.5 s in, longer than any idle-timeout or failover.
  • CHANGELOG stamped ## [5.1.0] - 2026-09-05 (website copy synced), with a Changed entry for the log level.

Verified

Build, unit suite, Postgres multi-node suite incl. the catch-up tests (which exercise exactly this path).

🤖 Generated with Claude Code

A lost LISTEN connection is routine -- a proxy idle-timeout, a failover, a
network blip -- and the node is resubscribed within a second, with catch-up
replaying whatever was missed. Logging it as an error with a stack trace fired
every alert on every idle-timeout, while the message itself promised a replay.
The warning names the cause; the error, with the exception, is reserved for
reconnects that keep failing, which is when someone should look.

CHANGELOG stamped for 5.1.0; Modgud runs beta.3 in production without findings
beyond this one.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@windischb
windischb merged commit a331b9f into develop Sep 5, 2026
12 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.

1 participant