Repository navigation
iroh broker: use the 7 self-hosted relay.cmux.dev relays - #7949
Conversation
…v URLs Replaces the 4 hosted iroh.link relays with our self-hosted fleet in both allowlists (web MANAGED_RELAY_URLS + presence worker APPROVED_IROH_RELAY_URLS, kept in lockstep) and the tests that referenced hosted URLs. The self-hosted relays run iroh-relay 1.0.2 behind per-region MIG+L4-LB (zero-downtime upgrades), gated by the cmux EdDSA JWT that /api/relay/token (merged, #7879) mints.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR moves the broker fleet to seven self-hosted relay.cmux.dev endpoints. The main changes are:
Confidence Score: 4/5The old-client relay publication path needs a compatibility plan before merging.
web/services/iroh/publicationPolicy.ts and workers/presence/src/routePrivacy.ts Important Files Changed
Reviews (1): Last reviewed commit: "iroh: point the broker relay fleet at th..." | Re-trigger Greptile |
| export const MANAGED_RELAY_URLS = [ | ||
| "https://euc1-1.relay.lawrence.cmux.iroh.link/", | ||
| "https://use1-1.relay.lawrence.cmux.iroh.link/", | ||
| "https://usw1-1.relay.lawrence.cmux.iroh.link/", | ||
| "https://aps1-1.relay.lawrence.cmux.iroh.link/", | ||
| "https://usc1.relay.cmux.dev/", | ||
| "https://usw1.relay.cmux.dev/", | ||
| "https://use4.relay.cmux.dev/", | ||
| "https://euw4.relay.cmux.dev/", | ||
| "https://apne1.relay.cmux.dev/", | ||
| "https://apse1.relay.cmux.dev/", |
There was a problem hiding this comment.
Old Relay Routes Lose Publication
When an already-deployed client continues reporting one of the four former relay URLs during rollout, the exact allowlist check now removes its relay hint. That device remains without a published relay path until it learns and reports the new fleet, so keep the old entries during a compatibility window or ensure clients migrate before this removal.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Per Aziz's ask: updates PR #7840's broker fleet from the 4 hosted iroh.link relays to the 7 self-hosted
relay.cmux.devURLs (usc1, usw1, use4, euw4, apne1, apse1, ape1), in both lockstep allowlists (web/services/iroh/publicationPolicy.ts+workers/presence/src/routePrivacy.ts) and the tests referencing hosted URLs.The self-hosted fleet runs iroh-relay 1.0.2 behind per-region MIG + L4 LB (zero-downtime upgrades, proven 180/180 probes during instance replacement), JWT-gated with per-endpoint connection caps + per-client bandwidth caps. Verified: web devices-route (25) + all 5 iroh suites (102) + presence worker (166) tests pass, web typecheck clean.
@azooz2003-bit merge this into #7840 when it looks good.
Need help on this PR? Tag
/codesmithwith what you need. Autofix is disabled.Note
Medium Risk
Changing the approved relay fleet affects which Iroh relay URLs clients can use after deploy; old lawrence.cmux.iroh.link URLs will no longer pass allowlist checks until endpoints report the new fleet.
Overview
Replaces the four hosted
lawrence.cmux.iroh.linkrelays with seven self-hostedrelay.cmux.devendpoints (usc1, usw1, use4, euw4, apne1, apse1, ape1) in the Iroh publication allowlists.web/services/iroh/publicationPolicy.ts(MANAGED_RELAY_URLS) andworkers/presence/src/routePrivacy.ts(APPROVED_IROH_RELAY_URLS) stay lockstep—only which relay URLs may be persisted or distributed changes; sanitization behavior is unchanged. Tests that used a hosted relay constant now point athttps://use4.relay.cmux.dev/.Reviewed by Cursor Bugbot for commit ecad500. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by cubic
Switch the iroh broker to our 7 self-hosted relays at
relay.cmux.devand update tests. Replaces the 4 hostediroh.linkrelays and keeps the web and presence allowlists in sync.web/services/iroh/publicationPolicy.tsandworkers/presence/src/routePrivacy.tsto the seven regions: usc1, usw1, use4, euw4, apne1, apse1, ape1.https://use4.relay.cmux.dev/.Written for commit ecad500. Summary will update on new commits.