search

mheyman / Isopoh.Cryptography.Argon2

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

Improved doc explanation for memoryCost #29

Closed LuckyWraptor closed 4 years ago

LuckyWraptor commented 4 years ago

This merge aims to improve and better explain the memoryCost documentation

65536 is not 64K but 64KB 64K (64,000) 1024 = 64MiB (65,536,000) 64KB (65536) 1024 = 64MB

This clears up any confusion and resolves issue #17

mheyman commented 4 years ago

Thanks! Changes available in v1.1.4. See (for instance) https://mheyman.github.io/Isopoh.Cryptography.Argon2/api/Isopoh.Cryptography.Argon2.Argon2.html#Isopoh_Cryptography_Argon2_Argon2_Hash_System_String_System_Int32_System_Int32_System_Int32_Isopoh_Cryptography_Argon2_Argon2Type_System_Int32_Isopoh_Cryptography_SecureArray_SecureArrayCall_