Expressで作ったAPIサーバで認証する

Question

ExpressでAPIサーバを作っています。認証のためにヘッダでkeyを受け取るのですが、次のようなコードを書いています：

app.get('/users', async function(req, res){
    const hash = crypto.createHash('sha256').update(req.header('key')).digest('hex')
    if(hash === ANSWER){
    ...
    }
}

これを全てのエンドポイントにコピペしていて効率が悪いです。
hash === ANSWERの比較を一か所に纏められませんか？

Answer 1

app.getの引数にいれてnext()すれば次に渡せたと思います。
未検証ですが

var hashCheck = function(req, res, next) {
    const hash = crypto.createHash('sha256').update(req.header('key')).digest('hex')
    if(hash === ANSWER){
      //次
      next()
    }
    else {
      //別ページ
      res.redirect('/betsupage');
    }
};
app.get('/betsupage', async function(req, res){

}

app.get('/', hashCheck)

app.get('/users', async function(req, res){

}
// 毎回エンドポイントに関数を差し込む下の感じでもよかったと思う
app.get('/users', hashCheck, async function(req, res){

}

リダイレクトする場合はチェックとリダイレクトの無限ループにならないように気を付けることになります。

Answer 2

app.all() であればすべてリクエストに対応できます。
詳細はAPIドキュメントを。