-
Notifications
You must be signed in to change notification settings - Fork 12
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
Switch to @noble/curves #25
Comments
Hi, what about providing a group interface (./src/group.ts) so we can plug different implementations of group, say sjcl and noble. |
@armfazh right now my fork has all calls to sjcl replaced with calls to curves. The interfaces (Scalar, Elt) are similar, however i've replaced calls to How should this be resolved? Also, should I continue? I don't want to continue if you aren't going to merge this. |
sjcl seems like piece of shit library, probably 20x slower than noble while having much more code |
I'm talking about this:
getting replaced with k: bigint. |
@armfazh OK, just understood what you've meant. You want to have voprf-ts not depend on any crypto library, right? This makes sense and could allow easy noble plug-in. However, where would you place noble-based group.ts though? Is it this repo? I don't want to place an abstraction into noble-curves directly because we have enough abstractions of our own and the new one would only be useful for voprf-ts. |
The PR like any other piece of code must pass through a code review process. So I cannot guarantee anything in advance. OTOH, I like the idea of using something that is faster, and also secure. The properties of sjcl are well-known. We can work together to get the changes needed to have noble as a plug-able lib. Feel free to email me for details. |
After our discussion: what's the decision? Should I pursue this, and if everything is ok after pr review, you would merge it; or you would prefer to keep sjcl? |
Please go for it, it's ok moving to noble. |
=> #26 |
I am potentially down to help with that |
I emailed you ;) |
Completed in #38 |
curves recently got out. This could be an opportunity to:
As a side note, you're using hash-to-curve which is also implemented in curves. We will probably publish an audit of the library some time soon.
I have started working on the pull request for voprf-ts, and now half-way there. Would you folks accept it?
The text was updated successfully, but these errors were encountered: