Promise 链未按预期顺序执行

Question

我正在尝试使用一系列承诺来验证用户名和密码，但这些承诺似乎解决得太早了。

我有来自这里涉及的两个不同文件的两种方法。

首先，我的控制器接收请求并调用我的模型进行身份验证。该模型应该返回一个承诺，该承诺将在它解析时具有身份验证结果。

这是我的控制器：

router.post('/api/auth-tokens', function (req, res, next) {
    var creds = {user: req.body.user, password: req.body.password};
    people.authenticate(creds)
    .then(function (result) {
      var status = result.valid ? 200: 401; 
      res.sendStatus(status);
    });
});

people 是词法范围模型。这是上面使用的方法的当前实现：

var bluebird = require('bluebird');
var bcrypt   = require('bcrypt');
var compare  = bluebird.promisify(bcrypt.compare);

exports.authenticate = function (args) {
    var promise = orm.select({
        db: db,
        table: table,
        where: {id: args.user},
        qrm: 'one'
    });

    promise.catch(function (err) {
        var auth_info = { valid: false };
        return auth_info;
    });

    promise.then(function (user) {
        var promise = compare(args.password, user.hash)
        .then(function (same) {
            var auth_info = { valid: same, permissions: user.permissions };
            return auth_info;
        });
        return promise;
    });

    return promise;
};

orm 返回一个promise，如果用户不存在则抛出错误，或者如果找到用户则解析并产生数据库行。 （这就是为什么我在那里有 promise.catch 调用。以防用户不存在）

如果用户确实存在，我想调用bcrypt.compare 将输入与数据库行中包含的哈希值进行比较。当异步完成时，我想解析承诺链并将控制权返回给控制器，返回 results 对象。

我遇到的问题是，我的控制器中的.then 正在立即执行，并且调用orm.select 返回了初始承诺的结果。谁能解释为什么会发生这种情况以及我该如何解决？

Answer 1

发生这种情况是因为您已将该回调直接连接到 orm.select 返回的承诺。相反，您需要使用 then 返回的承诺。

这是关于 Promise 的最重要的事情之一：then 返回一个新的 Promise。 （catch 也是如此。）如果您的处理函数从then 或catch 返回一个promise（技术上，任何thenable），那么then（或catch）的新promise ) 退货将根据您返回的承诺解决/拒绝。如果您的处理函数返回一个非承诺值，来自then（或catch）的新承诺将使用该值解析（而不是拒绝）。

所以authenticate 应该看起来像这样：

exports.authenticate = function (args) {
    return orm.select({
        db: db,
        table: table,
        where: {id: args.user},
        qrm: 'one'
    })
    .then(function (user) {
        return compare(args.password, user.hash)
        .then(function (same) {
            var auth_info = { valid: same, permissions: user.permissions };
            return auth_info;
        });
    })
    .catch(function (err) {
        var auth_info = { valid: false };
        return auth_info;
    });
};

请注意，更改既针对您的外部功能，也针对您在then 中所做的操作。另请注意，无论是来自orm.select 的原始承诺被拒绝，还是来自compare 的承诺，都将调用catch 回调中的代码。

要查看发生了什么，请将此 sn-p 与后面的进行比较：

// Returning the original promise
// Usually NOT what you want
function foo() {
  var promise = new Promise(function(resolve) {
    console.log("Starting first promise");
    setTimeout(function() {
      console.log("Resolving first promise");
      resolve("first");
    }, 500);
  });
  promise.then(function() {
    promise = new Promise(function(resolve) {
      console.log("Starting second promise");
      setTimeout(function() {
        console.log("Resolving second promise");
        resolve("second");
      }, 500);
    });
  });
  return promise;
}
foo().then(function(result) {
  console.log("Got result: " + result);
});

第二：

// Returning the original promise
// Usually NOT what you want
function foo() {
  return new Promise(function(resolve) {
    console.log("Starting first promise");
    setTimeout(function() {
      console.log("Resolving first promise");
      resolve("first");
    }, 500);
  })
  .then(function() {
    return new Promise(function(resolve) {
      console.log("Starting second promise");
      setTimeout(function() {
        console.log("Resolving second promise");
        resolve("second");
      }, 500);
    });
  });
}
foo().then(function(result) {
  console.log("Got result: " + result);
});

Answer 2

您的控制器执行的承诺与使用 bcrypt.compare 的承诺不同

这样做以确保 people.authenticate 返回一个承诺链，该链解析为比较后的内部结果：

var promise = orm.select(/* ... */)
    .then(/* bcrypt usage here */)
    .catch(/* error handling here */);

return promise;