-
Notifications
You must be signed in to change notification settings - Fork 465
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
Wishlist: Adding ciphers that are in other major libraries #346
Comments
I would like to note that Salsa20, XSalsa20 and ChaCha20 are all different ciphers. |
XSalsa20 is missing but the two others are already in the latest develop branch. Btw. do you intend to submit PR's for all these ciphers? Or is this really just a wishlist ;) |
SIV can be ticked as soon as #319 is finished |
@sjaeckel I have some basic C skills, really want to PR when I understand pointers and OO. |
@DonaldTsang I think we should aim at features available in modern crypto libraries like libsodium which is IMO what "cool kids" tend to use nowadays. If you have some spare time, could you please analyse what we are missing in comparison to libsodium? From your list I agree with |
@karel-m Should Salsa20 be extended to XSalsa20 (when longer IV is used) or should it be separate implementation? |
@LightBit if Salsa20 can easily be turned into XSalsa20 at runtime, so both could use the same basis, I'm pro extension. Otherwise we'd have to have a look what's best. |
ping @buggywhip ^^ |
@sjaeckel Due to salsa20_ivctr64(), extension would add 64 bytes overhead in state for Salsa20 to save key and IV. It would re-setup, if IV is different and longer than 8 bytes, else only counter would change. |
As |
yep
whoever wants to add xsalsa20 should just do it (preferably based on our salsa20) |
whoever wants to add xsalsa20 should just do it (preferably based on our salsa20)
we'll then bring it in form together if required :)
I'm willing to take this on but need to finish my current project. ...or at least get to a good stopping point. ...perhaps in a week or two?
|
@karel-m The reason I suggest adding legacy ciphers to the library is to allow future cryptographers to play around with it and test out new ideas. |
When talking about |
The goal of this Issue is to "price match" libtomcrypt with other cryptographic libraries in regards to functionality and variety.
reference: https://www.cryptopp.com/
reference: https://botan.randombit.net/manual/
reference: http://www.bouncycastle.org/csharp/index.html
Elliptic curves that are safe
reference: https://eprint.iacr.org/2013/647
The text was updated successfully, but these errors were encountered: