mosh-hamedani / vidly-api-node

307 stars 282 forks source link

Solution to those who are not able to POST 'users' data to the DB or the 'bcrypt' error. #43

Open harshit-jain13 opened 3 years ago

harshit-jain13 commented 3 years ago

Just install 'bcrypt' if not already or check if already installed. npm i bcrypt

Now, in your vidly-api-node folder goto routes folder and open up users.js in your code editor. and change this code (probably on line 21 & 22): const salt = await bcrypt.genSalt(10); user.password = await bcrypt.hash(user.password, salt); with this: const saltRounds = 10; const myPlaintextPassword = user.password; const bcrypt = require("bcrypt"); const salt = bcrypt.genSaltSync(saltRounds); const hash = bcrypt.hashSync(myPlaintextPassword, salt); user.password = hash; Save the file and you are good to go :)

dansalahi commented 3 years ago

Try to use the bcryptjs npm package.

harshit-jain13 commented 3 years ago

I tried but it did not work. I even tried using bcrypt-nodejs it didn't work either.

HafizSaifullah commented 3 years ago

Do not forget to rerun the script node index.js after changing your code. As I faced this issue.

the-black-viper commented 3 years ago

Fixed bcrypt error using bcrypt version 5.0.0