gleam-lang / crypto

⛓️ Crypto functionality for Gleam applications

Home Page:https://hexdocs.pm/gleam_crypto/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Suggestion: rename Crypto module to Sha

sporto opened this issue · comments

I suppose that evantually this library will have other algorithms like BCrypt and Argon2 (more suitable for password hashing).

But putting these in the Crypto module will be confusing. So it would make sense to have modules like Sha, Argon2, ... instead.

Consider renaming the current Crypto module to Sha or SecureHash
Thanks

I think that would be confusing given only one of the 6 functions in that module is for hashing. We would need to move the other elsewhere?

I'm also not sure we could implement bcrypt and argon2 without C extensions, if so we'd be unlikely to have them in this package.