Необработанное обещание Предупреждение: запрос на вытягивание NodeJS не работает

#node.js #mongodb #express #mern

Вопрос:

Я пишу следующий запрос, чтобы отменить подписку на пользователя. подписчики добавляются в базу данных в виде массива.

 router.put("/:id/unfollow",async (req,res) => {
    if(req.body.userId !== req.params.id){
        try{
            const user = await User.findById(req.params.id);
            const currentUser = await User.findById(req.params.id);
            if (user.followers.includes(req.body.userId)){
                await user.updateOne({$pull:{followers:req.body.userId}});
                await currentUser.updateOne({$pull:{followers:req.params.id}});
                res.status(200).json("user has been unfollowed");
            }else{
                res.send(403).json("you already unfollowed this user ");
            }

        }catch (err){
            res.status(500).json(err);
            console.log("err");
        }
    } else{
        res.status(403).json("You cant unfollow yourself")
    }
})
 

Когда запрос отправляется через postamn с правильным идентификатором пользователя, почтальон говорит «запрещено».

и терминал показывает следующую ошибку. Что здесь не так ?

 express deprecated res.send(status): Use res.sendStatus(status) instead routesuser.js:98:21
(node:15336) UnhandledPromiseRejectionWarning: Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
    at ServerResponse.setHeader (_http_outgoing.js:561:11)
    at ServerResponse.header (C:UsersBanchiDesktopReactTestmoodfix-api2node_modulesexpresslibresponse.js:771:10)
    at ServerResponse.send (C:UsersBanchiDesktopReactTestmoodfix-api2node_modulesexpresslibresponse.js:170:12)
    at ServerResponse.json (C:UsersBanchiDesktopReactTestmoodfix-api2node_modulesexpresslibresponse.js:267:15)
    at C:UsersBanchiDesktopReactTestmoodfix-api2routesuser.js:102:29
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:15336) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a prom
ise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#c
li_unhandled_rejections_mode). (rejection id: 1)
(node:15336) [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-z
ero exit code.
 

Комментарии:

1. Разве линия const currentUser = await User.findById(req.params.id); не должна быть const currentUser = await User.findById(req.body. userId) такой ? В своем фрагменте кода вы использовали req.params.id как в строке 4, так и в строке 5.

2. Ваша проблема возникла res.send(403).json("you already unfollowed this user "); , потому что send и json используют оба http.ServerResponse.write метода, и оба метода используются для отправки ответа клиенту. Правильный формат: res.status(403).json("you already unfollowed this user ");

Ответ №1:

внутри else блока из try внесите следующие изменения:

 try{
    ...
    if (user.followers.includes(req.body.userId)){
        ...
        res.status(200).json("user has been unfollowed");
    } else{
        // res.send(403).json("you already unfollowed this user ");
        //      ^^^ here is the error. It should be
        res.status(403).json("you already unfollowed this user ");
    }

} catch ...
 

Невозможно установить заголовки после их отправки клиенту

Это происходит, когда вы отправляете другой ответ за уже отправленным. И ваш код делает это. Сначала он отвечает send(403) , а затем json("you already unfollowed this user ") также отправляет.