Функция сброса пароля с помощью bcrypt не работает. Не удается установить заголовки после их отправки клиенту? - PullRequest
0 голосов
/ 24 октября 2018

Итак, я сделал функцию сброса пароля, и она выглядит следующим образом:

router.get('/reset/:id',function(req,res,next){
    User.findOne({'resetToken': req.params.id.trim()})
        .exec(function(error, user){
            if (error) return next(error);
            else if(!user){
                var err = new Error('Invalid Reset Link! The system is unable to find a user with the given reset code.');
                err.status = 401;
                return next(err);
            }else return res.render('resetpass',{token: req.params.id});
        })
})

router.post('/reset/:id', function(req,res,next){
    if(req.body.newPassword === req.body.confirmNewPassword)
    User.findOne({'resetToken': req.params.id.trim()},function(error,user){
        if(error) return next(error);
        bcrypt.hash(req.body.password, 5, function(err, hash) {
            if (err) {
                return next(err);
            }
            user.password = hash;
            user.resetToken ='';
            user.save(); 
        })

    });
    return res.redirect('/login');          
})

, поэтому, когда я заполняю поля пароля сброса и нажимаю "Отправить", я получаю это в консоли: Невозможно установить заголовки после их отправки вклиент

и если я использую это:

  router.post('/reset/:id', function(req,res,next){
    if(req.body.newPassword === req.body.confirmNewPassword)
    var cp;
    User.findOne({'resetToken': req.params.id.trim()},function(error,user){
        if(error) return next(error);
        bcrypt.hash(req.body.password, 5, function(err, hash) {
            if (err) {
                return next(err);
            }
            cp = hash;
        })
        user.password = cp;
        user.resetToken ='';
        user.save(); 
    });
    return res.redirect('/login');          
})

мой user.password пуст, и я получаю это: Невозможно установить свойство 'password' для null ->, потому что я установил его в UserSchema,Так почему это так?я попытался с console.logs в другой функции только с переменными и что-то вроде этого:

bcrypt.hash(.... {console.log('a') }) 
console.log('b');

моя консоль выглядит так:

b
a

Так что bcrypt console.log после.Что я могу сделать?

1 Ответ

0 голосов
/ 24 октября 2018

Обратите внимание, что User.findOne() является асинхронной операцией.

Перенаправление res.redirect('/login') должно называться после сохранения нового пароля.

Вы можете попробовать этокод ниже:

router.post('/reset/:id', function (req, res, next) {
    if (req.body.newPassword === req.body.confirmNewPassword) {
        User.findOne({
            'resetToken': req.params.id.trim()
        }, function (error, user) {
            if (error) return next(error);
            bcrypt.hash(req.body.password, 5, function (err, hash) {
                if (err) {
                    return next(err);
                }
                user.password = hash;
                user.resetToken = '';
                user.save();

                // Do the redirection here
                res.redirect('/login');
            })

        });
    }

    // Do nothing here
})
...