Skip to content

fix(p3-shim): align socket behavior with WASI tests#1484

Merged
andreiltd merged 2 commits into
bytecodealliance:mainfrom
andreiltd:fix/sockets-wasi
May 14, 2026
Merged

fix(p3-shim): align socket behavior with WASI tests#1484
andreiltd merged 2 commits into
bytecodealliance:mainfrom
andreiltd:fix/sockets-wasi

Conversation

@andreiltd
Copy link
Copy Markdown
Member

The fixes are driven by wasi upstream tests: allow wildcard binds, reject non-bindable addresses, and detect local bind conflicts for tcp and udp sockets.

The fixes are driven by wasi upstream tests: allow wildcard binds,
reject non-bindable addresses, and detect local bind conflicts for tcp
and udp sockets.
@andreiltd andreiltd requested a review from vados-cosmonic as a code owner May 14, 2026 11:18
vados-cosmonic
vados-cosmonic previously approved these changes May 14, 2026
Copy link
Copy Markdown
Collaborator

@vados-cosmonic vados-cosmonic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

Nothing but nits

Comment thread packages/preview3-shim/lib/nodejs/sockets/address.js Outdated
Comment thread packages/preview3-shim/lib/nodejs/workers/udp-worker.js
Comment thread packages/preview3-shim/lib/nodejs/workers/udp-worker.js Outdated
@andreiltd andreiltd enabled auto-merge May 14, 2026 15:11
@andreiltd andreiltd added this pull request to the merge queue May 14, 2026
Merged via the queue into bytecodealliance:main with commit 861a35e May 14, 2026
40 checks passed
@andreiltd andreiltd deleted the fix/sockets-wasi branch May 14, 2026 15: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.

2 participants