エラーコールバックからasync awaitへの書き換え

Question

Passport.jsを使ったログイン認証機能を実装する際に、エラーコールバックからasync awaitに書き換えたのですが、ログイン時にエラーとなってしまいます。（書き換え前は正常にログインできます）
間違っている箇所があればご指摘いただけると嬉しいです。

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

  req.login(user, (err) => {
    if (err) {
      res.status(400).send();
    } else {
      passport.authenticate('local')(req, res, () => {
        res.render('home');
      });
    }
  });
});

//async await
app.post('/login', async (req, res) => {
  const user = new User(req.body);

  try {
    await req.login(user);
    await passport.authenticate('local')(req, res, () => {
      res.render('home');
    });
  } catch (e) {
    res.status(400).send();
  }
});

###追記
エラーをログ出力したところ、req#login requires a callback functionとありました。
このような場合は書き換えできないということになるのでしょうか？

Comments

[gentaro]

エラーの内容ぐらいは書こうや。

[maskmelon]

ご指摘ありがとうございます。追記させていただきました。

Answer 1

間違っている箇所があれば

Express では ミドルウェア登録用関数（use(), post(), get(), etc...）に、async 関数を登録してはいけません。
通常のコールバック関数を登録してください。

promise を返却するコードを使う場合は、ミドルウェア内から参照できるスコープで async関数を宣言して実行するなどの工夫が必要です（middleware 内で async 関数を即時実行させるなど）。

---

Express は Promise が標準化される前のNodeJS用に作られたサーバーアプリケーション構築用フレームワークです。
ミドルウェア（引数に req, res, next が渡される専用のコールバック関数）を登録/逐次実行（フロー制御）する機構を提供してくれます。

v4 以降では 標準実装された Routerの機能がミドルウェア管理を担うように変わりましたが、
v3 以前で依存していた シンプルな実装だった　connect.js のソースから、上述の実装を確認できると思います。

Comments

[maskmelon]

回答ありがとうございます。Expressのミドルウェア関数はフロー制御を提供するための仕組みだったんですね。理解が深まりました。
特定のURLにアクセスした際に非同期処理を行う場合には、async関数の即時実行が一般的な書き方なのでしょうか？

[AkitoshiManabe]

申し訳ありません。「一般的かどうか」については分からないです。
ただ、私の場合、Express利用時はコールバック地獄を選んでいます。
（第三引数の next を活かし、続きの処理をミドルウェア化すれば良いため）。
また、async を使う場合、KoaJSを選んでいます（Expressに比べてマイナーですが）。

[maskmelon]

ミドルウェアについて理解を深めていこうと思います。
丁寧な回答ありがとうございました。