feat(mobile): support public secure tunnel pairing endpoints#8999
feat(mobile): support public secure tunnel pairing endpoints#8999LosEcher wants to merge 2 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds shared validation for bare Mobile addresses and 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 5cd2df0c-8e68-4bfb-9bc3-6b142a1927a9
📒 Files selected for processing (19)
docs/reference/2026-07-16-mobile-public-tunnel-pairing-design.mddocs/reference/mobile-public-tunnel.mdmobile/README.mdmobile/app/h/[hostId]/edit.tsxmobile/src/transport/host-endpoint-edit.test.tsmobile/src/transport/host-endpoint-edit.tsmobile/src/transport/pre-profile-pairing-coordinator.test.tsmobile/src/transport/rpc-client.test.tssrc/main/runtime/runtime-rpc.test.tssrc/renderer/src/components/mobile/NetworkInterfacePicker.tsxsrc/renderer/src/components/settings/MobilePairingSetupSection.test.tsxsrc/renderer/src/components/settings/MobilePairingSetupSection.tsxsrc/renderer/src/i18n/locales/en.jsonsrc/renderer/src/i18n/locales/es.jsonsrc/renderer/src/i18n/locales/ja.jsonsrc/renderer/src/i18n/locales/ko.jsonsrc/renderer/src/i18n/locales/zh.jsonsrc/shared/network/mobile-pairing-endpoint.test.tssrc/shared/network/mobile-pairing-endpoint.ts



Summary
Allow Orca Mobile pairing to use a user-managed secure WebSocket origin such as
wss://orca.example.comfor Cloudflare Tunnel and similar public TLS-terminated routes.ws://andwss://endpoints alongside existing IP, hostname, and host:port forms.wss://hostendpoints unchanged during host editing; use the scheme default (443) when the hostname changes.Screenshots
Three physical-device/UI screenshots are attached in the PR conversation (not committed to the repository, per project policy):
WebSocket openandReceived e2ee_readyover the publicwss://endpoint.Testing
pnpm lintpnpm typecheckpnpm test— 30,696 passed, 46 skippedpnpm build— root build passed in the recorded local evidence; not rerun after the final rebase.build:unpackreached an uncached Electron download and was stopped when it did not complete at a practical rate.AI Review Report
The review checked:
cloudflaredand does not assume a local-only origin.wss://hostcould rewrite its implicit port as6768.Security Audit
Notes