Passport.jsのログイン処理をPromiseで書き換えたい

Question

// エラーコールバック
router.post('/login', (req, res, next) => {
  const user = new User(req.body);

  req.login(user, (err) => {
    if (err) {
      next(err);
    } else {
      passport.authenticate('local')(req, res, () => {
        res.redirect('/');
      });
    }
  });
});

// Promise
router.post('/login', (req, res, next) => {
  const user = new User(req.body);

  req.login(user).then(passport.authenticate('local')(req, res, () => {
    res.redirect('/');
  }));
});

Node.jsを使ってアプリを作っており、ログイン認証にPassport.jsのローカルストラテジを使っています。ログイン処理の記述を上記のようにPromiseを使って書き換えてみたのですが、req#login requires a callback functionというエラーが発生しました。

書き換え方が間違っているのでしょうか？あるいはエラーコールバックにしか対応していないということも有り得るのでしょうか？（ドキュメントは上に書いたエラーコールバックの形で記述されています）

ご指摘いただけると幸いです。

Answer 1

従来の最終引数にコールバック関数を渡して実行してもらう非同期処理は、
勝手にPromiseになったりはしません。

値.thenは単純にthenプロパティを参照するという意味で、
Promiseインスタンスはthenプロパティをメソッドとして所持しているので、
実行出来るというだけの話です。

function hoge(cb) {
  // もしcallbackが存在するなら実行する
  if (cb) cb();
}
console.log(hoge());
// 返り値がないのでundefined

// undefinedのthenプロパティを閲覧しようとすると当然エラー
console.log(hoge().then);
// Uncaught TypeError: Cannot read property 'then' of undefined

// 実行するとPromiseインスタンスを作って返す関数を宣言
const piko = () => new Promise(resolve => {
  resolve()
})

// Promiseインスタンスを受け取ったら、それにはthenメソッドは存在するよね
console.log(piko().then);
// ƒ then() { [native code] }

piko().then(() => console.log(123));
// 123

---

能書きはわかった。
で、このPassport.jsとやらはPromise対応なのか？非対応なのか？

見ていきましょう。

PromiseはES2015で追加されました。
大昔から存在し、新しいものを瞬時に取り込むNode.jsにもかなり早い段階で導入されました。
対応リストで確認すると、0.12.18時点で多くの機能が利用可能になってるんですね。

対するPassport.jsはGitHubで公開されており、
プロジェクトの仕様はpackage.jsonを見れば分かるようになっています。

  "engines": {
    "node": ">= 0.4.0"
  },

enginesによると動作保証しているバージョンはNode.jsの0.4系、
0.12系より大幅に昔……

あっダメっぽい！

GitHubのIssueも見つけました。
Add support promises flow #536というまんまな要望があります。

ざっと読んだ感じ、2017年からPromise対応してくれーみたいな要望があがっているようですね。
うだうだ2-3年やり取りしてて、まだクローズなければマージされたプルリクもない。

結局音沙汰なしで放置かい。
非対応確定です。

---

結局どうすりゃええねん
Promiseベースでやりたければラッピングするしかないでしょうね。

passport.authenticate() using a Promise instead of a Custom Callback
という質問がスタックオーバーフローに存在しています。

これの回答がコールバックを無理やりPromiseに変換するライブラリでも噛ましたら？
const {promisify} = require("es6-promisify");はそれを目的としたライブラリの一部です。

es6というのはes2015という実装が決まるまでの仮の名前なので、
bluebirdを使ってラッピングした方が良いんじゃないですかね？

使い方はbluebird promiseみたいなワードで検索すれば出てくるんで頑張ってみてください。

Comments

[maskmelon]

調べ方まで教えていただきありがとうございます。
当たり前のようにPromiseを使っていましたが、ご指摘の通りPromiseを返す実装でなければ使えないのは当然ですね。
ライブラリについても参考になりました。