#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 ")
также отправляет.