NodeJS: UnhandledPromiseRejectionWarning при смене пароля

Question

Я работаю над изменением пароля в NodeJS, и во время обработки моего запроса я получаю следующую ошибку:

(node:16220) UnhandledPromiseRejectionWarning: Error: Can't set headers after they are sent.
    at validateHeader (_http_outgoing.js:491:11)
    at ServerResponse.setHeader (_http_outgoing.js:498:3)
    at ServerResponse.header (/home/pbaj/Documents/Projects/syberiaquotes-backend/node_modules/express/lib/response.js:771:10)
    at ServerResponse.send (/home/pbaj/Documents/Projects/syberiaquotes-backend/node_modules/express/lib/response.js:170:12)
    at ServerResponse.json (/home/pbaj/Documents/Projects/syberiaquotes-backend/node_modules/express/lib/response.js:267:15)
    at user.(anonymous function).updateOne.then.catch.err (/home/pbaj/Documents/Projects/syberiaquotes-backend/api/controllers/user.js:284:52)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)
(node:16220) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:16220) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

В моем маршруте changePassword я принимаю email,password,newPassword значений и:

1. проверка наличия электронной почты, если да ...

2. сравнение заданного пароля и пароля пользователя, если они совпадают ...

3. bcrypt.hash функция берет новый пароль, делает из него хеш и сохраняет его в mongoDB

Если я сообщу правильный адрес электронной почты и пароль, все пункты выше выполнены, но я получаюэто UnhandledPromiseRejectionWarning.

Я вижу, что причина этой проблемы заключается в функции user[0].updateOne(user[0]), именно в catch(), но я не знаю, что происходит.

// CHANGE USER PASSWORD
exports.changePassword = (req, res, next) => {
    User.find({ email: req.body.email })
    .then(user => {
        if (user.length < 1) {
            return res.status(401).json({
                message: 'Auth failed'
            })
        }

        const { password, newPassword } = req.body
        console.log(password, newPassword)
        bcrypt.compare(req.body.password, user[0].password, (err, result) => {
            console.log(result)
            if (err) {
                return res.status(401).json({
                    message: 'Error! Something goes wrong'
                })
            }
            if (result) {
                bcrypt.hash(newPassword, 10, (err, hash) => {
                    console.log(hash)
                    if (err) {
                        return res.status(500).json({
                            error: err
                        })
                    } else {
                        user[0].password = hash
                        user[0]
                        .updateOne(user[0])
                        .then(result => {
                            return res.status(200).json({
                                message: 'Password changed!',
                                result: result,
                            })
                        })
                        .catch(err => {
                            res.status(500).json({ message: err.message })
                        })
                    }
                })
            }
            return res.status(401).json({
                message: 'Auth failed'
            })
        })
    })
    .catch(err => {
        res.status(500).json({ error: err })
    })
}

Мне нужно получить [200]'Password changed!' ответ на мой запрос, но я получаю [401]'Auth failed' из-за UnhandledPromiseRejectionWarning.

Answer 1

Вы отправляете ответ 2 раза. Попробуйте этот код

 if (result) {
            bcrypt.hash(newPassword, 10, (err, hash) => {
                console.log(hash)
                if (err) {
                    return res.status(500).json({
                        error: err
                    })
                } else {
                    user[0].password = hash
                    user[0]
                    .updateOne(user[0])
                    .then(result => {
                        return res.status(200).json({
                            message: 'Password changed!',
                            result: result,
                        })
                    })
                    .catch(err => {
                        res.status(500).json({ message: err.message })
                    })
                }
            })
} else {
          return res.status(401).json({
            message: 'Auth failed'
          })
}

Answer 2

Я предполагаю, что вы используете Mongoose (из-за User.find(...)).

Если это так, вы не можете сделать user[0].updateOne(...), потому что user[0] уже является объектом типа User.

Вы или User.findOneAndUpdate(...), или user[0].save(...) после того, как вы установили новый пароль (что мне кажется более простым).