-
Notifications
You must be signed in to change notification settings - Fork 450
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: WebRTC-Direct support for Node.js #2583
base: main
Are you sure you want to change the base?
Conversation
Supports listening and dialing WebRTC Direct multiaddrs in Node.js. Depends on: - [ ] libp2p/go-libp2p#2827 - [ ] paullouisageneau/libdatachannel#1201 - [ ] paullouisageneau/libdatachannel#1204 - [ ] murat-dogan/node-datachannel#257 - [ ] murat-dogan/node-datachannel#256 Closes: - #2581
eb61d74
to
e2b6537
Compare
I read the code a bit, in this implementation when Peer B listens, it first listens for incoming stun packets using dgram, extracts the ufrag, and then establishes a connection with Peer A from another port. This works fine in a direct network connection. |
I haven't got to the point of testing NAT traversal yet, still a lot of PRs to land in the deps. I was wondering if each |
From the logic of the code, even though multiple RTCPeerConnections share a single UDP socket in UDPMUX mode, each of them manages their own status separately and there is no difference compared to regular RTCPeerConnections. This means that the negotiation process for them is also no different from regular RTCPeerConnections. |
6453a80
to
c2bc7fe
Compare
Supports listening and dialing WebRTC Direct multiaddrs in Node.js.
Depends on:
libp2p+webrtc+v1/
go-libp2p#2827DescriptionType
enum values murat-dogan/node-datachannel#259Closes:
Change checklist