Skip to content

Server docs: built-in Let's Encrypt, Cloudflare Tunnel, new portal sections - #20

Merged
epell-development merged 2 commits into
mainfrom
claude/project-thread-ghyg48
Oct 26, 2026
Merged

epell-development merged 2 commits into
mainfrom
claude/project-thread-ghyg48

Conversation

@epell-development

Copy link
Copy Markdown
Owner

Requested by epell · project thread

A follow-up to #14 that brings /server/docs/ up to date with the server features in #19.

Merge this after #19. The Pages workflow deploys the site as soon as this merges, so merging it first would put features on the site that the server doesn't have yet.

What changed

  • Options table: adds --acme, --acme-email, --acme-staging, --acme-http-port/--no-acme, --tunnel/--no-tunnel, --tunnel-token and --cloudflared.
  • HTTPS:
    • lists the three ways to get a certificate;
    • new section on the built-in Let's Encrypt (needs port 80, setcap on Linux, staging, --acme-http-port 0 when IIS or nginx already uses port 80);
    • certbot and win-acme files are now reloaded without a restart, so the restart hooks are gone.
  • New Cloudflare Tunnel section: setup with your own domain (token) or a quick trycloudflare.com address.
  • Web portal: describes the new sections (Studio, Library, Carts, Generator, Adverts, AzuraCast, Log), the phone layout and the update banner.
  • Reverse proxy:
    • keeping the server's HTTPS on now works behind a proxy that sends X-Forwarded-Proto, Forwarded, X-ARR-SSL or CF-Visitor;
    • the IIS steps no longer require turning HTTPS off;
    • the redirect-loop warning now says it affects servers older than 0.2.9.
  • Elsewhere:
    • port 80 added to the ports table;
    • Server settings list, troubleshooting rows for Let's Encrypt and the tunnel;
    • the broadcast-from-anywhere guide, the guide card and llms.txt updated to match.

docs/SERVER.md is left alone because #19 already changes it.

Testing

  • tools/build_site.py changes nothing on a second run.
  • tools/check_site.py passes: 17 pages, all links resolve.
  • Pages checked at 1280px and 390px with no horizontal scrolling.

Generated by Claude Code

…ctions

Documents the 0.2.9 server features from PR #19: --acme (automatic Let's
Encrypt with renewal, port 80, staging), --tunnel / --tunnel-token
(Cloudflare Tunnel via cloudflared), certificate files reloaded without a
restart, reverse proxies that send X-Forwarded-Proto / X-ARR-SSL /
CF-Visitor / Forwarded no longer loop, and the portal's Library, Carts,
Generator, Adverts and AzuraCast sections with the update banner.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014PpT9xjPyRV4m8YCwhW7if
@epell-development epell-development self-assigned this Oct 25, 2026

@mergeguards mergeguards Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MergeGuard review
Risk: 🟠 35/100 — moderate
Findings: 🟠 1 medium
Open Files changed for 1 inline review thread on the diff.

💡 Use the Fix All MergeGuard Findings checkbox comment on this PR to apply auto-fixes in batch, or reply @mergeguards fix on any inline finding.


Reviewed by MergeGuard AI

Comment thread site/server/docs/index.html Outdated
@mergeguards

mergeguards Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

MergeGuard agent actions

Fix all findings

All issues (1)
  • medium · site/server/docs/index.html:143 — Options table row for '--acme' contains an anchor link to '#acme' but the corresponding section heading with id='acme' is defined at line 184, which is later in the document. This forward reference…

Apply auto-fixes for the findings above on this branch:

  • Fix All MergeGuard Findings

Or reply with @mergeguards fix-all on this pull request.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014PpT9xjPyRV4m8YCwhW7if

@mergeguards mergeguards Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ MergeGuard — new commits are on this pull request.

Re-scan the latest diff: paste this line by itself (copy/paste is fine) — either on the Conversation tab (Add a comment) or on Files changed as a comment on the diff:

@mergeguard-followup

@mergeguards mergeguards Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MergeGuard review
Risk: 🟠 35/100 — moderate
Findings: 🟠 1 medium
Open Files changed for 1 inline review thread on the diff.

💡 Use the Fix All MergeGuard Findings checkbox comment on this PR to apply auto-fixes in batch, or reply @mergeguards fix on any inline finding.


Reviewed by MergeGuard AI

Comment thread site/server/docs/index.html
@epell-development
epell-development merged commit 256b511 into main Oct 26, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants