Skip to content

feat(infra): wire loopover.ai as a live secondary domain#5345

Merged
JSONbored merged 1 commit into
mainfrom
loopover/wire-domain
Jul 12, 2026
Merged

feat(infra): wire loopover.ai as a live secondary domain#5345
JSONbored merged 1 commit into
mainfrom
loopover/wire-domain

Conversation

@JSONbored

Copy link
Copy Markdown
Owner

Summary

  • Adds loopover.ai (UI Worker) and api.loopover.ai (API Worker) as custom-domain routes, live side by side with the existing gittensory.aethereal.dev domains -- no cutover, no redirect yet.
  • Introduces PUBLIC_SITE_ORIGIN_ALIASES (comma-separated, config-driven) so the post-GitHub-OAuth returnTo redirect works correctly from either domain. Without it, normalizeReturnTo's allowlist only ever contained the single PUBLIC_SITE_ORIGIN value, so a login started on loopover.ai would authenticate fine but bounce back to the old domain on completion.

Test plan

  • npm run typecheck
  • npm run test:ci (full local gate, green)
  • New unit test covering PUBLIC_SITE_ORIGIN_ALIASES: primary origin still works, an aliased origin is accepted verbatim, a second comma-separated alias works, an untrusted origin still falls back, and unset-alias behavior is unchanged from before this var existed
  • Post-merge: verify loopover.ai and api.loopover.ai resolve over TLS once the custom domains provision, and complete a real GitHub OAuth sign-in from loopover.ai end-to-end

Closes #4762.

Adds loopover.ai (UI) and api.loopover.ai (API) as custom-domain routes
alongside the existing gittensory.aethereal.dev domains, live side by
side with no cutover yet. Introduces PUBLIC_SITE_ORIGIN_ALIASES so the
post-GitHub-OAuth-login redirect works correctly from either domain --
without it, a login started on loopover.ai would authenticate but
bounce back to the old domain, since PUBLIC_SITE_ORIGIN only ever
supported a single allowed origin.
@JSONbored JSONbored self-assigned this Jul 12, 2026
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
gittensory-ui e7142fa Commit Preview URL

Branch Preview URL
Jul 12 2026, 02:23 PM

@codecov

codecov Bot commented Jul 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.36%. Comparing base (fb76770) to head (e7142fa).
⚠️ Report is 10 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5345      +/-   ##
==========================================
- Coverage   94.40%   94.36%   -0.04%     
==========================================
  Files         550      550              
  Lines       44116    44118       +2     
  Branches    14631    14632       +1     
==========================================
- Hits        41649    41634      -15     
- Misses       1792     1809      +17     
  Partials      675      675              
Flag Coverage Δ
shard-1 43.75% <0.00%> (-0.26%) ⬇️
shard-2 34.04% <100.00%> (-0.61%) ⬇️
shard-3 32.13% <100.00%> (+0.58%) ⬆️
shard-4 31.28% <0.00%> (-0.14%) ⬇️
shard-5 33.04% <0.00%> (-0.12%) ⬇️
shard-6 43.63% <0.00%> (+0.27%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/auth/github-oauth.ts 95.04% <100.00%> (+0.10%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JSONbored
JSONbored merged commit b779f28 into main Jul 12, 2026
19 checks passed
@JSONbored
JSONbored deleted the loopover/wire-domain branch July 12, 2026 14:30
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.

Wire loopover.ai as a live secondary domain

1 participant