search

mheyman / Isopoh.Cryptography.Argon2

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

Strip Base64 padding #18

Closed DarkCoderX closed 5 years ago

DarkCoderX commented 5 years ago

The Argon2.Hash(password); method creates a hash string that contains B64 padding in salt and password hash. The hash formatting spec does not allow this.

The padding cause interoperability problems with stricter implementations like node-argon2

mheyman commented 5 years ago

Thanks for pointing this out - I was unaware that the base64 encoding was non-standard.

mheyman commented 5 years ago

It turns out, at one point I was aware of it because there are comments in the base64 decoding code about it. Well, now it is fixed and I've pushed a new version to nuget.org.