Express用户注册接口捕获500错误后用户仍被创建的原因

问题原因 #

你的代码中，用户创建（await user.save()）操作发生在JWT生成之前，500错误实际是在生成JWT的步骤中抛出的。具体流程：

1. await user.save()执行完成后，用户数据已经成功写入数据库；
2. 后续调用jwt.sign时如果出现异常（比如jwtSecret配置错误、签名逻辑故障），回调里的throw err会被外层catch块捕获，返回500状态码，但此时用户已经被创建完成。

另外注意：catch块里的console.error('err.message');存在错误，引号会导致只打印字符串err.message，无法看到真实错误信息，不利于排查问题。

解决方案 #

1. 先排查JWT生成失败的具体原因 #

修正catch块的日志代码，查看真实错误：

catch(err) {
    console.error(err.message); // 去掉引号，打印真实错误信息
    res.status(500).send('Server Error');
}

运行后查看控制台输出，就能明确JWT生成失败的具体诱因（比如配置项缺失、密钥格式错误等）。

2. 确保用户创建与JWT生成的原子性（可选） #

如果希望JWT生成失败时不保留用户数据，可以将jwt.sign改为Promise版本，用await调用，在出错时回滚已创建的用户：

try {
    // ... 此前的用户检查、实例创建、密码加密、保存逻辑 ...

    // 用Promise封装jwt.sign回调
    const token = await new Promise((resolve, reject) => {
        jwt.sign(
            payload,
            config.get('jwtSecret'),
            { expiresIn: 360000 },
            (err, token) => {
                if (err) reject(err);
                else resolve(token);
            }
        );
    });

    res.json({ token });
} catch(err) {
    console.error(err.message);
    // 若错误发生在用户保存后，尝试删除已创建的用户
    if (user && user._id) {
        await User.findByIdAndDelete(user._id);
    }
    res.status(500).send('Server Error');
}

3. 改用JWT的Promise API（推荐） #

如果你的jsonwebtoken版本在v8及以上，可以直接使用官方提供的Promise版本，无需手动封装：

// 替换原有的jwt.sign回调写法
const token = await jwt.sign(
    payload,
    config.get('jwtSecret'),
    { expiresIn: 360000 }
);
res.json({ token });

这样整个流程都处于try块的异步操作中，任何错误都会被捕获，你也可以根据需求在catch中执行用户数据回滚。

内容的提问来源于stack exchange，提问作者tukitooo1998