Question

我有一个简单的模块，其中包含几个与Active Directory交互的功能。

使用标准回调样式，代码可以正常工作：

/**
* mylib.js
**/

const ActiveDirectory = require('activedirectory');

// Instantiate AD client
const ad = new ActiveDirectory({
  url: 'ldaps://...',
  baseDN: 'DC=...',
});

module.exports = {
  // Authenticate user against the AD
  authenticate: (username, password, callback) => {
    // Authentication request
    ad.authenticate(username, password, (err, status) => {
      callback(err, status);
    });
  },
};

/**
* client.js
**/

const mylib = require('./mylib');

mylib.authenticate('<username>', '<password>', (err, status) => {
  if (err) {
    console.log(`Error: ${err}`);
    return;
  }
  console.log(`Success: ${status}`);
});

执行结果：

> node client.js
Success: true

下一步是重构我的lib方法以使用Promises而不是回调：

/**
* mylib.js
**/

const ActiveDirectory = require('activedirectory');
const Promise = require('bluebird');

//...

module.exports = {
  // Authenticate user against AD
  authenticate: Promise.method((username, password) => {
    ad.authenticate(username, password, (err, status) => {
      if (err) throw err;
      return status;
    });
  }),
};

/**
* client.js
**/

const mylib = require('./mylib');

myLib.authenticate('<username>', '<password>').then((status) => {
  console.log(`Success: ${status}`);
}).catch((err) => {
  console.log(`Error: ${err}`);
});

执行结果：

> node client.js
Success: undefined

所以看起来status没有得到解决。

如果我将AD服务器URL更改为其他内容（强制连接错误并查看拒绝），我可以看到解析和拒绝日志：

> node client.js
Success: undefined

/Users/.../mylib.js:84
      if (err) throw err;
               ^

Error: connect ECONNREFUSED <IP>
    at Object.exports._errnoException (util.js:1036:11)
    at exports._exceptionWithHostPort (util.js:1059:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1080:14)

我做错了什么？无法理解。

Answer 1

你期望Promise.method()过多，抛出的错误或返回的值确实会被包裹，但那不是你在做什么 - 你是从ad.authenticate投掷/返回＆＃39; s回调。

你想要的是ad.authenticate的宣传版，蓝鸟非常简单。

module.exports = {
  // Authenticate user against AD
  authenticate: Promise.promisify(ad.authenticate)
};

Node.js：使用Bluebird将模块函数从回调转换为promises

1 个答案: