fix: force https via redirectscheme (+ comment) #6

Closed
autonomic-bot wants to merge 0 commits from forceSsl into main
Owner

Replace the deprecated Traefik v1 headers.SSLForceHost/headers.SSLHost middleware — which is a no-op on Traefik v2+ (cctest runs v3.6.11) — with a working redirectscheme (http→https, 301 permanent), and correct the now-inaccurate comment.

Empirically verified on cctest (Traefik v3.6.11): old labels never redirected EXTRA_DOMAINS→DOMAIN (alias served 200 directly); new middleware forces https identically. No behavioural regression; removes dead config.

Supersedes the stale, unmergeable upstream coop-cloud/custom-html#5.

Replace the deprecated Traefik **v1** `headers.SSLForceHost`/`headers.SSLHost` middleware — which is a **no-op on Traefik v2+** (cctest runs v3.6.11) — with a working `redirectscheme` (http→https, 301 permanent), and correct the now-inaccurate comment. Empirically verified on cctest (Traefik v3.6.11): old labels never redirected EXTRA_DOMAINS→DOMAIN (alias served 200 directly); new middleware forces https identically. No behavioural regression; removes dead config. Supersedes the stale, unmergeable upstream coop-cloud/custom-html#5.
autonomic-bot added 1 commit 2026-08-03 18:16:43 +00:00
Author
Owner

!testme

!testme
Author
Owner

🌻 cc-cicustom-html @ 67faf3f3 failurehttps://drone.ci.commoninternet.net/recipe-maintainers/cc-ci/1146

(summary card unavailable — see the run for details.) full logs · dashboard

<!-- cc-ci:testme --> 🌻 **cc-ci** — `custom-html` @ `67faf3f3` ❌ **failure** → https://drone.ci.commoninternet.net/recipe-maintainers/cc-ci/1146 _(summary card unavailable — see the run for details.)_ [full logs](https://drone.ci.commoninternet.net/recipe-maintainers/cc-ci/1146) · [dashboard](https://ci.commoninternet.net/)
Author
Owner

!testme

!testme
Author
Owner

🌻 cc-cicustom-html @ 67faf3f3 passed

cc-ci result card

level

full logs · dashboard

<!-- cc-ci:testme --> 🌻 **cc-ci** — `custom-html` @ `67faf3f3` ✅ **passed** [![cc-ci result card](https://ci.commoninternet.net/runs/1148/summary.png)](https://drone.ci.commoninternet.net/recipe-maintainers/cc-ci/1148) [![level](https://ci.commoninternet.net/runs/1148/badge.svg)](https://drone.ci.commoninternet.net/recipe-maintainers/cc-ci/1148) [full logs](https://drone.ci.commoninternet.net/recipe-maintainers/cc-ci/1148) · [dashboard](https://ci.commoninternet.net/)
Author
Owner

!testme

!testme
autonomic-bot added 1 commit 2026-08-03 20:46:41 +00:00
autonomic-bot changed title from fix: SSLForceHost/SSLHost -> redirectscheme to fix: force https via redirectscheme (+ comment) 2026-08-03 20:50:29 +00:00
autonomic-bot force-pushed forceSsl from 3ac7819ff5 to 2dfbbaf645 2026-08-03 20:50:30 +00:00 Compare
Author
Owner

!testme

!testme
Author
Owner

!testme

!testme
autonomic-bot force-pushed forceSsl from 2dfbbaf645 to dcc24f8c15 2026-08-03 20:56:15 +00:00 Compare
Author
Owner

Auto-closed by /recipe-upgrade: its changes are already in upstream main (merged upstream); mirror main re-synced

Auto-closed by /recipe-upgrade: its changes are already in upstream main (merged upstream); mirror main re-synced
autonomic-bot closed this pull request 2026-08-03 20:59:48 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: recipe-maintainers/custom-html#6