CI valideert het productie-Caddyfile niet — caddy validate toevoegen aan test.yml (follow-up #20) #26

New Issue

rve · 2026-07-12T18:54:43Z

rve commented

2026-07-12 18:54:43 +00:00

Context (follow-up van #20)

De frontend-crash uit #20 kon door CI glippen omdat test.yml het testimage bouwt met Caddyfile.test (COPY Caddyfile.test /etc/caddy/Caddyfile) — het échte productie-Caddyfile wordt in geen enkele check geraakt. Een parse-fout daarin komt pas aan het licht als de deploy-gate faalt, en dan is de kapotte container al live (recreate: always) → downtime. Detectie is er sinds #20, preventie nog niet.

Fix

caddy validate van beide Caddyfiles als eerste stap in de test-job (caddy:2-alpine, zelfde base-image als de Dockerfile). Een ongeldige config faalt daarmee al in de PR-check, in seconden, vóór er iets gebouwd of gedeployed wordt. Dit is exact de check waarmee de #20-bug lokaal werd aangetoond, nu structureel in de pipeline.

.github/workflows/ staat in PROTECTED.md; wijziging is expliciet geautoriseerd door de product owner in deze opdracht.

Verificatie

PR-run: test-job groen mét de nieuwe validate-stap (bewijst de positieve flow op de echte runner)
Negatieve flow lokaal al bewezen in #20: dezelfde validate faalt exact op de toenmalige fout

## Context (follow-up van #20) De frontend-crash uit #20 kon door CI glippen omdat `test.yml` het testimage bouwt met `Caddyfile.test` (`COPY Caddyfile.test /etc/caddy/Caddyfile`) — het échte productie-`Caddyfile` wordt in geen enkele check geraakt. Een parse-fout daarin komt pas aan het licht als de deploy-gate faalt, en dan is de kapotte container al live (recreate: always) → downtime. Detectie is er sinds #20, preventie nog niet. ## Fix `caddy validate` van **beide** Caddyfiles als eerste stap in de test-job (`caddy:2-alpine`, zelfde base-image als de Dockerfile). Een ongeldige config faalt daarmee al in de PR-check, in seconden, vóór er iets gebouwd of gedeployed wordt. Dit is exact de check waarmee de #20-bug lokaal werd aangetoond, nu structureel in de pipeline. `.github/workflows/` staat in PROTECTED.md; wijziging is expliciet geautoriseerd door de product owner in deze opdracht. ## Verificatie - [ ] PR-run: test-job groen mét de nieuwe validate-stap (bewijst de positieve flow op de echte runner) - [x] Negatieve flow lokaal al bewezen in #20: dezelfde validate faalt exact op de toenmalige fout

rve referenced this issue from a commit

2026-07-12 19:01:37 +00:00

ci: validate both Caddyfiles in the test job (#26)

rve referenced a pull request that will close this issue

2026-07-12 19:02:14 +00:00

CI: caddy validate van beide Caddyfiles in de test-job #28

rve closed this issue

2026-07-12 19:04:30 +00:00

rve referenced this issue from a commit

2026-07-13 07:13:20 +00:00

ci: validate the production Caddyfile inside the built image (#26)

rve referenced a pull request that will close this issue