search

gleam-lang / crypto

⛓️ Crypto functionality for Gleam applications
https://hexdocs.pm/gleam_crypto/
Apache License 2.0
35 stars 10 forks source link

Suggestion: rename Crypto module to Sha #5

Closed sporto closed 2 years ago

sporto commented 2 years ago

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

lpil commented 2 years ago

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.