Skip to content
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

You know "Salt"? #17

Closed
rugk opened this issue Jan 16, 2017 · 3 comments
Closed

You know "Salt"? #17

rugk opened this issue Jan 16, 2017 · 3 comments
Labels

Comments

@rugk
Copy link

rugk commented Jan 16, 2017

Do you know https://github.com/devi/Salt?

Please tell me how horrible it is if you think so.
Apart from that (which you would have done anyway, even if I had not requested this) you might (re)use some code from there or might suggest all users to switch to your lib (when it is finished).
In any case I wanted to make you aware of this lib.

@paragonie-scott
Copy link
Member

Salt seems fine, except:

  1. They don't transparently fall back to the PHP extension if it's available.
  2. They use multiple classes in a single file.
  3. They don't include a lot of nice libsodium APIs e.g. crypto_sign_detached() and crypto_box_seal().
  4. They're not on Packagist.

@rugk
Copy link
Author

rugk commented Jan 16, 2017

And…

  1. this

Anyway I think it is not so good. It seems to be quite unmaintained…

@paragonie-scott
Copy link
Member

Right, I don't think there's much maintenance going into Salt. If it's ok with you, I'm going to close this issue. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants