Skip to content

fix: use URL polyfill for websocket bridge#99

Merged
V3RON merged 3 commits intomainfrom
fix/url
Apr 7, 2026
Merged

fix: use URL polyfill for websocket bridge#99
V3RON merged 3 commits intomainfrom
fix/url

Conversation

@V3RON
Copy link
Copy Markdown
Contributor

@V3RON V3RON commented Apr 7, 2026

Summary

  • replace the runtime websocket bridge's custom absolute-URL parser with the standard URL API via react-native-url-polyfill
  • keep rejecting non-network dev server values while preserving the full host, including explicit ports and bracketed IPv6 addresses
  • extend the focused runtime tests to cover hostname ports plus IPv6 cases with and without user info

Testing

  • pnpm exec vitest run packages/runtime/src/client/getWSServer.test.ts

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
react-native-harness Ready Ready Preview, Comment Apr 7, 2026 9:56am

Request Review

@V3RON V3RON changed the title fix: replace runtime URL parsing for websocket bridge fix: use URL polyfill for websocket bridge Apr 7, 2026
@V3RON V3RON merged commit cc06526 into main Apr 7, 2026
6 checks passed
@V3RON V3RON deleted the fix/url branch April 7, 2026 15:48
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