search

mheyman / Isopoh.Cryptography.Argon2

Fully managed .Net Core implementation of Argon2
Other
196 stars 9 forks source link

Argon2.Verify has no option to set the secret, so custom implementation needed #10

Closed Karsten-W closed 6 years ago

Karsten-W commented 6 years ago

Please add some tests that include a secret.

mheyman commented 6 years ago

You are right - optional secrets should be easy. I believe the lack of a helper function that takes a secret to be a mistake because that should probably be a fairly common use case - as you realized, custom code was needed if you wanted to use secrets as part of the hash.

I've pushed the updates and version 1.0.8 of https://www.nuget.org/packages/Isopoh.Cryptography.Argon2 has the changes.