search

shaneMangudi / bcrypt-nodejs

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

Example not working #18

Open grasGendarme opened 11 years ago

grasGendarme commented 11 years ago

The example in README doesn't work and throws an error

var hash = bcrypt.hashSync("bacon");
Error: data and salt arguments required
hugotown commented 9 years ago

Wow after 2 years i get the same error... bcrypt has support ? someone can fix that ?

var hash = bcrypt.hashSync("bacon"); Error: data and salt arguments required

mayman99 commented 8 years ago

same here

mogita commented 8 years ago

Same here, on calling bcrypt.genSalt()

cjduncana commented 8 years ago

Hello @mogita: Today, I just did a pull request that fixes this problem. It seems, though, that this repository is not being updated. The last commit was almost a year ago. I suggest that, in the meantime, you use my version of the repository: npm install --save https://github.com/cjduncana/bcrypt-nodejs.git. When @shaneGirish accepts my pull request, I'll let you know so that you can go back t this repository.

netojose commented 7 years ago

2017 and the error still exists :/

nukeop commented 6 years ago

A year later and it's still there