Applied to the live Caddyfile (snapshot: ~/Caddyfile.pre-news-public) and reloaded; ddns_update published the A record and Let's Encrypt issued a cert. The block moved out of the "Internal only -- no public DNS record" section into the Public section rather than just gaining a marker in place. Those section headers are load-bearing documentation, and a public block sitting under an internal header would make the file lie about its own security posture. /api/feedback/* bypasses Authelia because ntfy runs action buttons from the phone and a native app cannot complete an interactive login -- the exemption ntfy, Immich and Jellyfin already have on this box. That path is HMAC-signed per (cluster, kind), so a leaked URL casts one vote on one story. Verified after reload: / and /healthz return 403 from Authelia while /api/feedback/... returns 404, and a 404 from the still-running old backend is what proves Caddy routed it through unauthenticated. Worth recording, since it is irreversible: publishing the name also publishes it to Certificate Transparency, which is append-only and world-readable. The hostname is public forever regardless of any later change.
139 lines
5.2 KiB
Markdown
139 lines
5.2 KiB
Markdown
# Deploying
|
|
|
|
This replaces the `news-backend` service (from `../news_triage`) in place: same
|
|
host, same vhost, same ntfy topic.
|
|
|
|
## 1. Compose
|
|
|
|
Replace the `news-backend` block in `~/compose.yml`:
|
|
|
|
```yaml
|
|
news-backend:
|
|
image: git.rcjohnstone.com/connor/news:latest
|
|
restart: unless-stopped
|
|
networks:
|
|
- internal
|
|
environment:
|
|
- TZ=America/Louisville
|
|
- NEWS_DB=/data/news.db
|
|
- NEWS_CONFIG_DIR=/config
|
|
- NTFY_URL=${NTFY_URL}
|
|
# The old service defaulted to this topic (bin_support.rs:121) rather than
|
|
# setting it, so reusing it keeps the existing phone subscription working.
|
|
- NTFY_TOPIC=news-triage
|
|
- NTFY_USER=${NTFY_USER}
|
|
- NTFY_PASS=${NTFY_PASS}
|
|
# REQUIRED for feedback buttons. ntfy runs http actions FROM THE PHONE, so
|
|
# this must be the public URL, not a container address. Its absence is
|
|
# exactly why the previous version's buttons never fired.
|
|
- NEWS_API_BASE_URL=https://news.rcjohnstone.com
|
|
- NEWS_FEEDBACK_SECRET=${NEWS_FEEDBACK_SECRET}
|
|
# One LLM call per day, for the digest prose only.
|
|
- NEWS_LITELLM_KEY=${NEWS_LITELLM_KEY}
|
|
volumes:
|
|
# `U` makes podman chown the volume to the container's uid. The image runs
|
|
# as uid 10001, not root, so without it SQLite fails with
|
|
# "attempt to write a readonly database".
|
|
- ./data/news2:/data:U
|
|
- ./config/news2:/config:ro
|
|
healthcheck:
|
|
# NOT curl: the v0.2 image shipped none, so its healthcheck failed forever
|
|
# while the service was fine and `podman ps` reported it unhealthy for days.
|
|
test: ["CMD", "python", "-c",
|
|
"import urllib.request,sys; sys.exit(0 if urllib.request.urlopen('http://localhost:3000/healthz',timeout=4).status==200 else 1)"]
|
|
interval: 60s
|
|
timeout: 5s
|
|
retries: 3
|
|
start_period: 20s
|
|
```
|
|
|
|
Copy `profile/*.toml` to `~/config/news2/`. A bind-mounted config edit needs
|
|
`sudo podman restart news-backend`; `podman-compose up -d` will not restart a
|
|
container when only a mounted file changed. An **environment** change does need
|
|
the full `up -d`.
|
|
|
|
## 2. Caddy — APPLIED 2026-09-08
|
|
|
|
Done. The block was moved out of the "Internal only" section into the Public
|
|
section (the section headers are load-bearing documentation, and leaving a
|
|
public block under an internal header would make the file lie), given a
|
|
`# ddns: public` marker, and reloaded. `ddns_update` published the A record and
|
|
Let's Encrypt issued a cert for the name.
|
|
|
|
Snapshot of the previous file: `~/Caddyfile.pre-news-public`.
|
|
|
|
Verified after reload: `/` and `/healthz` return 403 (Authelia gating), while
|
|
`/api/feedback/...` returns 404 -- a 404 from the OLD backend, which proves
|
|
Caddy routed it through without authentication.
|
|
|
|
```
|
|
# ddns: public
|
|
news.rcjohnstone.com {
|
|
import secure_headers
|
|
import accesslog
|
|
|
|
# ntfy executes a notification's action buttons FROM THE PHONE, and a native
|
|
# app cannot complete an interactive login -- the same reason ntfy, Immich
|
|
# and Jellyfin are exempted from the identity gateway on this box. So this
|
|
# one path bypasses Authelia. It is safe to: every request carries an HMAC
|
|
# signed over (cluster_id, kind), so a leaked URL casts one vote on one
|
|
# story rather than acting as a bearer token for the model.
|
|
@feedback path /api/feedback/*
|
|
handle @feedback {
|
|
reverse_proxy news-backend:3000
|
|
}
|
|
|
|
# Everything else is browser-facing, so it goes behind Authelia. Tapping the
|
|
# digest notification opens a browser, where an interactive login works.
|
|
handle {
|
|
import authelia
|
|
reverse_proxy news-backend:3000
|
|
}
|
|
}
|
|
```
|
|
|
|
The change from the current block is deliberate and worth stating plainly: the
|
|
vhost moves from **LAN-only** (`tls internal` + `import internal`, which aborts
|
|
anything outside 10.0.0.0/24, 10.0.1.0/24, 10.89.0.0/16) to **public**. Without
|
|
that, feedback buttons fail on cellular and the digest link is unopenable away
|
|
from home -- and silent failure there is precisely how the previous version
|
|
ended up with 32 labels in a week.
|
|
|
|
`/srv/news/dist` is now unused and the root-owned `news-update` systemd timer
|
|
that populated it should be disabled:
|
|
|
|
```sh
|
|
sudo systemctl disable --now news-update.timer
|
|
```
|
|
|
|
**Current state:** the vhost is public and correctly gated, but `news-backend`
|
|
is still the old Rust service, which no longer has its SPA served by Caddy. So
|
|
`https://news.rcjohnstone.com/` is a 404 behind the login until the compose
|
|
service is switched to this image. Notifications from the old service are
|
|
unaffected.
|
|
|
|
## 3. Scheduling
|
|
|
|
```
|
|
*/15 * * * * podman exec news-backend python -m news.cli poll
|
|
*/20 * * * * podman exec news-backend python -m news.cli notify
|
|
45 8 * * * podman exec news-backend python -m news.cli digest
|
|
```
|
|
|
|
## 4. Verifying it actually works
|
|
|
|
```sh
|
|
podman exec news-backend python -m news.cli poll # fetch, embed, cluster
|
|
podman exec news-backend python -m news.cli simulate # what WOULD be sent
|
|
podman exec news-backend python -m news.cli notify --dry-run
|
|
```
|
|
|
|
Then the one test that matters, because it is the loop that was broken before:
|
|
send a real notification, **tap a feedback button on the phone**, and confirm a
|
|
row lands in `feedback`:
|
|
|
|
```sh
|
|
podman exec news-backend python -c \
|
|
"import sqlite3;print(sqlite3.connect('/data/news.db').execute('select * from feedback').fetchall())"
|
|
```
|