Question

我有一个Express.js路由，可以通过用户名和密码从MongoDB数据库中检索用户。我的目标是从请求正文中获取用户名和密码，通过用户名找到用户，然后验证密码是否匹配。如果密码匹配，我想返回用户对象和200状态代码，如果密码不匹配，我想返回带有错误消息的404。

当前，我的路线正在运行，但是即使密码不匹配，也总是会返回用户。我正在检查它们是否匹配并抛出错误，但似乎并没有像我期望的那样抛出错误。理想情况下，如果服务呼叫中的密码不匹配，我想抛出一个错误，并在路由中捕获该错误以返回正确的状态代码。

我试图在路线中捕获错误并从服务中返回被拒绝的诺言，但仍然返回了用户。我的直觉是findOne接受的回调在返回记录后运行了吗？

我在控制台中收到一个UnhandledPromiseRejectionWarning，如果密码不匹配，该行对应于我抛出错误的行：

app_1            | GET /user 200 221 - 4.297 ms
app_1            | (node:257) UnhandledPromiseRejectionWarning: Error: Password does not match
app_1            |     at /usr/src/app/services/userService.ts:17:37
app_1            |     at Generator.next (<anonymous>)
app_1            |     at fulfilled (/usr/src/app/services/userService.ts:5:58)

server.ts

注意我将导入时将find重命名为findUser find as findUser

app.get('/user', authenticateToken, async (req: Request, res: Response): Promise<void> => {
    const user = await findUser(req.body.username, req.body.password);
    if (!user) {
      res.status(404).send();
      return;
    }
    res.status(200).send({ user });
});

userService.ts

const find = (username: string, password: string) => {
    return User.findOne({ username }, async (err: Error, user: any) => {
        if (err) throw err;
        const passwordMatches = await user.validatePassword(password);
        if (!passwordMatches) throw Error("Password does not match")
    });
}

user.ts（用户猫鼬模型）

import mongoose from 'mongoose';
import bcrypt from 'bcrypt';

interface User {
  username: string;
  password: string;
  isModified: (string: any) => boolean;
}
const SALT_WORK_FACTOR = 10;
const userSchema = new mongoose.Schema({
  username: { type: String, required: true, index: { unique: true } },
  password: {
    type: String,
    unique: false,
    required: true
  }
},
{ timestamps: true }
);

userSchema.pre('save', function(next) {
  const user = this as unknown as User;
  if (!user.isModified('password')) return next();
  bcrypt.genSalt(SALT_WORK_FACTOR, (err: any, salt: any) => {
      if (err) return next(err);

      bcrypt.hash(user.password, salt, function(err: any, hash: any) {
          if (err) return next(err);

          user.password = hash;
          next();
      });
  });
});

userSchema.methods.validatePassword = function (pass: string) {
  return bcrypt.compare(pass, this.password);
};

const User = mongoose.model('User', userSchema);

export { User };

Answer 1

在我的userService中，我更改了与findOne一起查询以解决此问题的方式。我从阅读Does mongoose findOne on model return a promise?

时得到了这个主意

const find = (username: string, password: string) => {
  return User.findOne({ username }).exec().then((user: any) => {
    const passwordMatches = user.validatePassword(password);
    if (!passwordMatches) throw Error("Password does not match")
    return user;
  });
}

猫鼬返回文档，即使抛出错误

1 个答案: