-
Notifications
You must be signed in to change notification settings - Fork 10
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
UTP required? #16
Comments
It’s required for any kind of hole punching to work but we could make it optional. What’s your usecase? |
I'm testing dropping this module into firefox, using libdweb for TCP and UDP sockets. The
Option 1 would probably be impossible, as we'd have to convince Mozilla to take the patch. 3 is not ideal, as then firefox peers would be less connectable. 2 seems viable, but I'm not sure how complex it would be to implement. |
We have a wasm version actually called utp-wasm we could fall back to. It’s feature complete but much slower than the native one |
Seems like utp-wasm could be a good replacement, though I doesn't seem to be working with libdweb at the moment. Do you know a good debug/test setup for utp? |
The issue with |
Is the utp transport mandatory for this library to work? Unlike
discovery-swarm
this module does not gracefully handle an error when importingutp-native
. This is an issue for platforms where we cannot use the native bindings that the native utp implementation needs.If utp is required, is there a javascript implementation of the protocol that could be used on top of UDP sockets which I could substitute for the native implementation?
The text was updated successfully, but these errors were encountered: