Add ntfy / Web Push notification channel #72
Labels
No labels
area:ai
area:ci-cd
area:notifications
area:observability
area:public-pages
backlog
bug
duplicate
enhancement
help wanted
invalid
question
type:decision
type:feature
type:infra
type:maintenance
type:security
v1.0.1
v1.1.0
v1.2.0
v1.3.0
v2.0.0
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
rbrooks/WeatherBot#72
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The notifier architecture is one-file-per-channel; ntfy is the de-facto self-hosted push standard and would give phone push without Twilio costs or Discord/Matrix infrastructure. Missed because the channel list tracks what the owner already ran. Effort S, value High for any phone user.
Filed from the 2026-07-17 codebase audit (
docs/.internal/report-2026-07-17.md), finding I-02.Done in #112 (merged). ntfy channel implemented end-to-end:
ChannelType.ntfy+ migration 0025,notifiers/ntfy.py(POST to{server_url}/{topic}with Title/Priority/Tags, Bearer or Basic auth, severity→priority default, Transient/Permanent outbox error classification, RFC-2047 header encoding),NtfyConfigvalidation + secret redaction, both dispatch chains wired, and full web-form UI inchannels.html. Like email/pushover/sms it carries NWS alerts + all-clears (the SPC/forecast/AI dispatch chains are discord+webhook-only by design). 20 notifier tests; migration 0025 verified on a real-DB copy.Web Push (VAPID keys + service worker + subscription management) is intentionally not in this — it's a substantially larger effort. If wanted, file it fresh; ntfy already delivers the phone-push value this issue was about.