Default salt size and iterations
haythem opened this issue · 4 comments
haythem commented
First of all i want to thank you for this great and clear implementation.
I wanted you just to know that there is no point in having large salts. 16 bytes salt is sufficient. Also 100000 is good if you're using it locally, on a server it will cost a lot. From what i read, in a production environment, you should use 20000 iterations.
shawnmclean commented
Awesome, thanks for the information. I'll push an update soon with the changes reflecting this suggestion.
Terebi42 commented
The hashing number changes over time. 100k is the recommended 2016 number.
shawnmclean commented
Hi @Terebi42, it seems the default Hash Iteration was already 100k. https://github.com/shawnmclean/SimpleCrypto.net/blob/master/src/PBKDF2.cs#L23