Error data and hash arguments required

I’ve done some investigation using console.log() statements (not proud of it) but I think I’ve managed to find out the issue. If we add in the the first console log statement here:

  app.post('/register', checkNotAuthenticated, async (req, res) => {
    try {
      console.log("BCRYPT COMPARE RUNS HERE")
      const hashedPassword = await bcrypt.hash(req.body.password, 10)
      const newUser = new User({
        id: Date.now().toString(),
        name: req.body.name,
        email: req.body.email,
        password: hashedPassword
      })
      res.redirect('/login')
      console.log(newUser)
    } catch {
      res.redirect('/register')
    }

and the second one here:

 const initializePassport = require('./passport-config')
  initializePassport(
    passport,
    email => User.find({email: email}).then((result) => { console.log("USER DATA EXTRACTED HERE") }).catch((err) => { console.log(err) }),
    id => User.find({id: id})
  )

The next time you click on login, you should see an output like:

Listening on port 3000
BCRYPT COMPARE HAPPENING
Error: data and hash arguments required
...
...
...
USER DATA EXTRACTED HERE

Notice that bcrypt.compare is being run before we are actually able to grab the user information from the DB? This means that all the arguments into that function are null, which is what is returning that error. Now, I’m no JS expert, but this can be fixed with an await statement added here:

   function initialize(passport, getUserByEmail, getUserById) {
      const authenticateUser = async (email, password, done) => {
        const user = await getUserByEmail(email)
        if (user === null) {
          return done(null, false, { message: 'No user with that email' })

        }

Which makes sure that the user info is queried from the DB before moving along in the script.

Any ideas why I’m getting the error

(node:21964) UnhandledPromiseRejectionWarning: Error: data and hash arguments required

on line

const password = await bcrypt.compare(req.body.password);

full code:

const User = require('../models/user')
const bcrypt = require('bcrypt')
const register_index = (req, res) => {
res.render('auth/register', {title: 'Register'})
}
const register_store = async (req, res) => {
try {
const hashedPassword = await bcrypt.hash(req.body.password, 10)
const user = new User({
first_name: req.body.first_name,
surname: req.body.surname,
email: req.body.email,
username: req.body.username,
addnumber: req.body.addnumber,
addstreet: req.body.addstreet,
addpostcode: req.body.addpostcode,
contact: req.body.contact,
password: hashedPassword,
})
user.save();
res.status(201).send();

} catch(error) {
console.log(error)
res.send(500).send();
}
}
const login_index = (req, res) => {
res.render('auth/login', {title: 'login'})
}
const login_auth = async (req, res) => {
const email = req.body.email;
const password = await bcrypt.compare(req.body.password);
const user = User.findOne({
email: email,
password: password,
})
try {
if(!user) {
return res.status(400).send("We don't have a user with that email")
} else {
res.status(200).send('Success')
}
} catch(error) {
res.status(500).send('Server Error');
}

}

module.exports = {
register_index,
register_store,
login_index,
login_auth,
}

Когда вы вызываете функцию bcrypt.compare (), вам необходимо убедиться, что вы указали и пароль, и хешированный пароль. Так что положите туда распечатку и посмотрите, что вам не хватает.

Случай 1: req.param (‘пароль’) по какой-то причине пуст (имя поля в вашей html-форме может содержать опечатку)

Случай 2: user.encryptedPassword пуст. В этом случае причина может заключаться в том, что вы не шифруете свой пароль при создании / обновлении своей пользовательской записи. Чтобы убедиться, что вы это делаете, откройте свою базу данных и после создания нового пользователя проверьте запись в БД и посмотрите, есть ли там ваш encryptedPassword. Вот пример того, как вы можете зашифровать пароль во время создания / обновления:

В вашей модели User у вас могут быть следующие функции:

beforeUpdate: function (values, next) {
    if (values.password) {
        bcrypt.hash(values.password, 10, function (err, hash) {
            if (err) {
                return next(err);
            }
            values.encryptedPassword = hash;
            next();
        });
    }
},

beforeCreate: function (values, next) {
    bcrypt.hash(values.password, 10, function (err, hash) {
        if (err) {
            return next(err);
        }
        values.encryptedPassword = hash;
        next();
    });
}

Обратите внимание на эту строку:

values.encryptedPassword = hash;

Поскольку у вас нет атрибута пароля в схеме, предоставленный файл values.password не будет нигде сохранен. Другая вещь в вашем случае может заключаться в том, что вы нигде не устанавливаете encryptedPassword. Таким образом, другой подход — изменить вашу схему и переименовать encryptedPassword в простой пароль, а затем заменить

values.encryptedPassword = hash;

С участием

values.password = hash;

I have a user log system that works well, instead when comparing the hash of the database with the introduction in a html field throws me the following error

Error: data and hash arguments required

I am sending 3 parameters just as it says in the documentation, which is the password in plain text, the hash and a callback. Thank you in advance

'use strict';
const bcrypt = require('bcrypt')
module.exports = (sequelize, DataTypes) => {

const User = sequelize.define('User', {

email: {

allowNull: false,

type: DataTypes.STRING,

unique: true

},

password_hash: DataTypes.STRING,

password: DataTypes.VIRTUAL

}, {});
User.login = (email, password) => {

return User.findOne({

where: {

email: email

}

}).then(user => {

if(!user) return null

return user.authenticatePassword(password).then(valid => valid ? user : null)

})

}
/* =============== /

/ METODO PARA COMPARAR HASH /

/ =============== */
User.prototype.authenticatePassword = (password) => {

return new Promise((res, rej) => {

bcrypt.compare(password, this.password_hash, (err, valid) => {

if(err) return rej(err)
  res(valid)
})

})

}



  


                       
  




User.associate = function(models) {

// associations can be defined here

};

User.beforeCreate((user, options) => {

return new Promise((res, rej) => {

if(user.password) {

bcrypt.hash(user.password, 10, (err, hash) => {

user.password_hash = hash

res()

})

}

})

})

return User;

};