watcher: startup notification no longer says every endpoint works (closes #230) #241

Merged
clawbot merged 1 commits from issue-230-startup-notification-text into next 2026-10-02 08:42:35 +02:00
Collaborator

Closes #230

The startup notification sent when DNSWATCHER_SEND_TEST_NOTIFICATION=true
ended "All notification channels are working." The message is written once and
handed to every notification endpoint before any delivery has succeeded or
failed, so that claim was never checked, and it was false whenever one endpoint
refused the message. Its last line now says only that it is a test notification
sent to every configured notification endpoint.

The startup notification test now compares the whole message, not just the
title and priority. No names are configured in that test, so every count in the
message is 0.

Model: opus-5-5

Closes https://git.eeqj.de/sneak/dnswatcher/issues/230 The startup notification sent when `DNSWATCHER_SEND_TEST_NOTIFICATION=true` ended "All notification channels are working." The message is written once and handed to every notification endpoint before any delivery has succeeded or failed, so that claim was never checked, and it was false whenever one endpoint refused the message. Its last line now says only that it is a test notification sent to every configured notification endpoint. The startup notification test now compares the whole message, not just the title and priority. No names are configured in that test, so every count in the message is 0. Model: opus-5-5
clawbot added the needs-review label 2026-10-02 08:17:19 +02:00
clawbot self-assigned this 2026-10-02 08:17:19 +02:00
Author
Collaborator

Review passed on a08f407.

Model: opus-5-5

Review passed on a08f407. Model: opus-5-5
clawbot added 1 commit 2026-10-02 08:42:20 +02:00
The startup notification ended "All notification channels are working.",
but it is written once and handed to every notification endpoint before
any delivery has succeeded or failed, so the claim was never checked and
was false whenever one endpoint refused it. It now says only that it is a
test sent to every configured notification endpoint. The startup
notification test checks the whole message.

Model: opus-5-5
clawbot force-pushed issue-230-startup-notification-text from a08f407dc8 to ddcb0dca07 2026-10-02 08:42:20 +02:00 Compare
clawbot merged commit 5db5048754 into next 2026-10-02 08:42:35 +02:00
clawbot deleted branch issue-230-startup-notification-text 2026-10-02 08:42:35 +02:00
clawbot removed the needs-review label 2026-10-02 08:42:36 +02:00
Sign in to join this conversation.