-
Notifications
You must be signed in to change notification settings - Fork 17
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
Dropping TweetNaCl #18
Comments
I didn't realize that we actually have at least three providers:
I took the various chloride-* modules and pulled their histories together to create one (much smaller) repository that uses Sodium-Universal, but unfortunately there are a few functions missing. More info here: sodium-friends/sodium-javascript#19 Here's my branch: https://github.com/christianbundy/chloride/tree/sodium-universal Compatibility:
|
@arj03 What are you relying on in the SSB browser demo? |
Browser core is very dependant on sodium-browserify to have web assembly working. JS is insanely slow compared to web assembly. So I don't really care about tweetnacl. I would actually like to get rid of tweetnacl as that makes the bundle smaller. So big +1 from me :-) Great work @christianbundy. And thanks for the ping @staltz I didn't have notifications turned on for this repo until now. |
Sweet. I'm curious to benchmark libsodium-wrappers (emscripten) vs sodium-javascript (handwritten) once all of the methods are ported. If we can just depend on sodium-universal that'd remove lots of complexity from this module and consolidate our libsodium dependencies to the sodium-friends org. |
I ran sodiumperf against libsodium-wrappers and sodium-javascript and it looks like this might actually give us a nice performance boost: (This assumes that my PRs into sodium-javascript are merged, and currently leaves out |
Have read this, but haven't got super familiar with details. Please tag me if you need my attention on this. FWIW I'm mainly using graphql to connect from browser to back end, so don't use eg ssb-client which needs this much. |
🎉 All of our required functionality has been released in Sodium-Javascript! |
Issues like #3 make me wonder whether this module would be better as a wrapper around sodium-chloride and sodium-universal, dropping support for TweetNaCl entirely. Trying to bridge two disparate APIs into a consistent interface is hard work, and unless we feel very strongly about TweetNaCl vs Sodium-JavaScript I think this might be the right move.
The text was updated successfully, but these errors were encountered: