ExpressJSミドルウェアリダイレクトが多すぎます

Question

私の/appルートで2つのミドルウェアを使用しようとしていますが、そのルートではユーザー認証とそのアカウントの状態を確認します。私は両方のミドルウェアを用意していますが、req.session.accountStatusが私がそれを提供した条件と等しくない場合、私は無限のリダイレクトに遭遇しています。一般的に、私はリダイレクトされているページへのアクセス権のみをユーザに強制しようとしています。ミドルウェアを間違った方法で使用していますか？より良いアプローチがありますか？

function isLoggedIn(req, res, next) { 
    if (req.isAuthenticated()){ 
     return next(); 
    } 
    res.redirect('/login'); 
} 

function accountStatus(req, res, next) { 
    if(req.session.accountStatus == "active" || req.session.accountStatus == "trialing"){ 
     return next(); 
    } else { 
     //Endless loop. Need to fix 
     res.redirect('/app/settings/billing'); 
    } 
} 

router.use(require('./site-routes')); 
router.use('/app', isLoggedIn, accountStatus, require('./app-routes')); 

Answer 1

おそらくミドルウェアをapp-router.jsに移動する方が簡単です。

だからあなたのメインのファイルは、これだけにします：

router.use('/app', require('./app-routes')); 

はapp-routes.jsでは、最初に「開く」でなければなりませんURLのルートを追加します。ルートの残りの部分が続く

router.use(isLoggedIn, accountStatus); 

：制限ミドルウェアが続く

router.get('/settings/billing', ...); 

。

このように、/app/settings/billingのリクエストは、ミドルウェアをまったく受け継がれず、リダイレクトループも発生しません。

isLoggedInの場合は/appで始まるルートは、あなたが同様の方法でそれを使用することができますのために必須である：

router.use(isLoggedIn); 
router.get('/settings/billing', ...); 
router.use(accountStatus); 
router.get(...);