search

shaneMangudi / bcrypt-nodejs

Native implementation of bcrypt for NodeJS
Other
574 stars 69 forks source link

Documentation says that Progress callback isn't required, but it is. #39

Open dasebast opened 9 years ago

dasebast commented 9 years ago

So if you could change that, it would be nice.

Save other users the trouble that I had figuring that error out.

cesarmalari commented 9 years ago

hash's 3rd argument (the one labelled progress) isn't required - it can be null. However, you can't just pass the callback as the third arg (the api node.bcrypt.js uses) and expect it to work.

dasebast commented 9 years ago

Oh, I see.

Thanks for the info, I'll change my code around tomorrow.