-
Notifications
You must be signed in to change notification settings - Fork 1.2k
WebTransport WASM support #3285
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
base: master
Are you sure you want to change the base?
Conversation
…alentlessguy/go-libp2p into webtransport-wasm
This will be a lot of maintenance burden. I'd like to gauge interest from some other parties before merging this and maintaining it going forward. |
Looking at #3281 @MarcoPolo is interested in this. |
@talentlessguy - Thank you for this, this is really cool! To get this merged, a few things that would give more confidence here is to add more tests here and add support for this in the interop test runner (libp2p/test-plans). cc @MarcoPolo @sukunrt who will review when time permits |
I can help add to the interop test runner too. (Maybe we can pair on it?) |
what kind of tests? GOOS=js go test or something like e2e? |
There's a transport interop test suite here: https://github.com/libp2p/test-plans/tree/master/transport-interop that you can add to test Wasm WebTransport against other implementations |
I'm not sure how to run interoperability tests from that repo. can someone from the team help? also FYI it works just fine in chrome but on FF it fails during dial I think. Seems like FF is missing certain APIs |
Implements WebAssembly support for the WebTranport transport based on #2191
I've also added an example to demonstrate connecting to a remote libp2p WT node, it works in Chrome
live demo: https://bafybeie2gvjhno6djfyohmmubhfi7qtpidli7w2rqpryomd4yq7us6auiq.ipfs.dweb.link/