Question

在快递+护照+本地策略应用中，我使用bcrypt对密码进行哈希处理，这是可行的：

var bcrypt = require('bcrypt-nodejs');

familySchema.pre('save', function(next) {
  var family = this;
  var SALT_FACTOR = 14;

  if (!family.isModified('password')) return next();

  bcrypt.genSalt(SALT_FACTOR, function(err, salt) {
    if (err) return next(err);

    bcrypt.hash(family.password, salt, null, function(err, hash) {
      if (err) return next(err);
      family.password = hash;
      next();
    });
  });
});

然后我使用promisify和async/await进行重构：

const bcrypt = require('bcrypt-nodejs');
const util = require('util');
const bcryptGenSalt = util.promisify(bcrypt.genSalt);
const bcryptHash = util.promisify(bcrypt.hash);

familySchema.pre('save', async function(next) {
  var family = this;
  const SALT_FACTOR = 14;
  if (!family.isModified('password')) return next();
  const salt = await bcryptGenSalt(SALT_FACTOR).catch(next);
  const hash = await bcryptHash(family.password, salt, null).catch(next);
  family.password = hash;
  next();
});

这种重构真的正确吗？
如何重新检查bcryptGenSalt或bcryptHash中的错误是否被正确捕获？是否可以通过某种方式“强制” bcryptGenSalt引发错误以进行测试？
下一步，如何使用.catch(next)实用程序功能删除两个wrapAsync：

wrapAsync.js：

module.exports = fn => (req, res, next) => fn(req, res, next).catch(next);

以下尝试不起作用，错误：family.isModified()不是一个函数，（可能是因为this不再正确）。而如何处理wrapAsync的参数，因为next应该是第三个参数？

familySchema.pre(
  'save',
  wrapAsync(async function(req, res, next) {
    var family = this;
    const SALT_FACTOR = 14;
    debugger;
    if (!family.isModified('password')) return next();
    const salt = await bcryptGenSalt(SALT_FACTOR);
    const hash = await bcryptHash(family.password, salt, null);
    family.password = hash;
    next();
  })
);

Answer 1

重构不正确，因为在.catch语句之后，该函数的其余部分将继续运行。因此，例如，如果bcryptGenSalt引发错误，则会调用next（由于.catch(next)），但它还会继续下一行代码，直到函数结束（其中{ {1}}再次被调用。）

通常，在next函数中，在可能引发错误的语句周围使用async：

try/catch

是否可以通过某种方式“强制” familySchema.pre('save', async function(next) { const SALT_FACTOR = 14; if (!this.isModified('password')) return next(); try { const salt = await bcryptGenSalt(SALT_FACTOR); const hash = await bcryptHash(this.password, salt, null); this.password = hash; return next(); } catch(err) { return next(err); } });引发错误以进行测试？

这取决于您用于测试的工具，但是有sinon之类的程序包可以存入现有功能，以便您可以控制地制造然后抛出错误，然后可以对其进行测试。

使用有希望的方法重构护照的本地策略。 .catch（）

1 个答案: