shaneMangudi / bcrypt-nodejs

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

Throw real errors instead of strings #34

Open rrrnld opened 9 years ago

rrrnld commented 9 years ago

I would suggest throwing real errors instead of strings because they provide additional information helpful for debugging. This appears to be a rather minor change with a decent benefit and if there's. interest in this I can write the code and open the pull request. Any thoughts?

fpirsch commented 9 years ago

This project is long dead. I suggest using live forks, like https://github.com/fpirsch/twin-bcrypt or https://github.com/dcodeIO/bcrypt.js