Skip to content

site: sign-in covers the music too (merge with the 0.2.0-beta25 release) - #483

Draft
jeffcrouse wants to merge 1 commit into
mainfrom
site/media-needs-sign-in
Draft

jeffcrouse wants to merge 1 commit into
mainfrom
site/media-needs-sign-in

Conversation

@jeffcrouse

Copy link
Copy Markdown
Member

Hold until the 0.2.0-beta25 release. The site deploys to Cloudflare on any push to main that touches site/, and until a server release ships #482, the current copy ("Sign-in covers the library, not the music") is the true one.

Rewrites the "Listening away from home" note and the reverse-proxy warning on index.html, and the FAQ's reverse-proxy answer, for ADR-0176 Phase 4:

  • The note now says sign-in covers the library and the music, and a speaker gets an expiring link. It keeps the recommendation to stay private, on the reasons that still hold: plain HTTP, servers older than 0.2.0-beta25 serve music without sign-in, and servers from before October 2026 may have none.
  • The reverse-proxy warning now reads "a reverse proxy alone does not make Familiar safe to publish", for the same reasons, plus the existing point that a proxy login locks out the apps and speakers.
  • docs/SITE-CLAIMS.md: both rows are rewritten and re-verified against the Phase 4 code, and the ADR-0145 expiry note records that the media trigger fired.
  • site/e2e/toggle.spec.ts pins the new wording and that "not the music" is gone.

If the release isn't named beta25, change the three version mentions.

Site Playwright suite: 25 passed. scripts/check_docs.py: OK.

🤖 Generated with Claude Code

"Sign-in covers the library, not the music" becomes false with the release
that enforces media credentials (#482). The recommendation, keep the server
private, now stands on plain HTTP and on older servers; the reverse-proxy
warning keeps its conclusion with those reasons. The ledger rows and the
ADR-0145 expiry note are rewritten to match, and the site e2e test pins the
new wording.

Merge with the 0.2.0-beta25 release, not before: the site deploys on push.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant