Express+MongoDB删除接口内部服务器错误问题排查求助

我尝试用Postman调用API删除MongoDB中的用户文档，更新功能运行正常，但删除功能返回内部服务器错误，请求协助排查。

我的Express代码： #

const router = require("express").Router();
const User = require("../models/User");
const bcrypt = require("bcrypt");
//update
router.put("/:id", async (req, res) => {
    if ((req.body.userId === req.params.id) || req.body.isAdmin) {
        if (req.body.password) {
            try {
                const salt = await bcrypt.genSalt(10);
                req.body.password = await bcrypt.hash(req.body.password, salt);
            }
            catch (err) {
                return res.status(500).json(err);
            }
        }
        try {
            const user = await User.findByIdAndUpdate(req.params.id, {
                $set: req.body,
            });
           return res.status(200).json("Account has been updated");
        }
        catch (err) {
            return res.status(500).json(err);
        }
    }
    else return res.status(400).json("You can only update your account!!!");
});
//delete
router.delete("/:id", async (req, res) => {
    if ((req.body.userId === req.params.id) || req.body.isAdmin) {
        try {
           await User.deleteOne(req.params.id);
            return res.status(200).json("Account has been deleted");
        }
        catch (err) {
            return res.status(500).json(err);
        }
    }
    else return res.status(400).json("You can only update your account!!!");
});


module.exports = router;

Postman请求情况： #

发送DELETE请求至对应ID的接口，返回500 Internal Server Error。

核心错误点： #

1. deleteOne()参数错误：Mongoose的deleteOne()方法需要接收查询条件对象，而非直接传入ID字符串。你直接传req.params.id会导致Mongoose无法解析查询规则，触发内部错误。
2. 提示文案错误：权限验证不通过时的提示写了"update"，不符合删除接口的语义。

修复方案（二选一即可）： #

方案一：修正deleteOne()的参数格式

//delete
router.delete("/:id", async (req, res) => {
    if ((req.body.userId === req.params.id) || req.body.isAdmin) {
        try {
           await User.deleteOne({ _id: req.params.id }); // 传入查询对象指定_id
            return res.status(200).json("Account has been deleted");
        }
        catch (err) {
            console.log(err); // 建议打印错误日志，方便排查
            return res.status(500).json(err);
        }
    }
    else return res.status(400).json("You can only delete your account!!!"); // 修正提示文案
});

方案二：使用更贴合代码风格的findByIdAndDelete()

该方法直接接收ID参数，和你更新接口用的findByIdAndUpdate()用法一致，代码更统一：

//delete
router.delete("/:id", async (req, res) => {
    if ((req.body.userId === req.params.id) || req.body.isAdmin) {
        try {
           await User.findByIdAndDelete(req.params.id); // 直接传入ID即可
            return res.status(200).json("Account has been deleted");
        }
        catch (err) {
            console.log(err); // 建议打印错误日志，方便排查
            return res.status(500).json(err);
        }
    }
    else return res.status(400).json("You can only delete your account!!!"); // 修正提示文案
});

内容的提问来源于stack exchange，提问作者Robert Downey Jr.