2016-06-24 18 views
-1

私は、bin/wwwファイルとapp.jsファイルを持つexpress-generatorで生成されたアプリケーションを持っています。 app.jsに次のコードを追加しました:ノード/ Express 4ミドルウェアが動作しない

app.use(function(req, res, next){ 
     console.log("--------new middleware"); 
     next(); 
    }); 

しかし、ミドルウェアはまったく呼び出されていません。アプリの残りの部分は正常に動作し、サーバーが稼働しており、要求を受け取っていることを確認しました。ここで

はapp.jsです:

var express = require('express'); 
var path = require('path'); 
var favicon = require('serve-favicon'); 
var logger = require('morgan'); 
var cookieParser = require('cookie-parser'); 
var bodyParser = require('body-parser'); 
var session = require('express-session'); 
var passport = require('passport'); 


var routes = require('./routes/index'); 
var users = require('./routes/users'); 

//ルートパス

app.use(function(req, res, next){ 
    console.log("--------new middleware"); 
    next(); 
}); 

// app.use(auth.isLoggedIn); 

// catch 404 and forward to error handler 
app.use(function(req, res, next) { 
    var err = new Error('Not Found'); 
    err.status = 404; 
    next(err); 
}); 

// development error handler 
// will print stacktrace 
if (app.get('env') === 'development') { 
    app.use(function(err, req, res, next) { 
    res.status(err.status || 500); 
    res.render('error', { 
     message: err.message, 
     error: err 
    }); 
    }); 
} 

// production error handler 
// no stacktraces leaked to user 
app.use(function(err, req, res, next) { 
    res.status(err.status || 500); 
    res.render('error', { 
    message: err.message, 
    error: {} 
    }); 
}); 

module.exports = app; 
+0

ミドルウェアの関数の署名が間違っています。 'err'引数はありません。上記の私のポストによるとhttp://expressjs.com/en/4x/api.html#app.use – jfriend00

+0

を参照してください。私もそれを試してみました(..私も正面に間違いなくこれを試しました)。私はそれをもっと明確にすべきです。 – Mankind1023

+0

サーバは稼動していますか?何かリクエストがありますか? 'app.get( '/'、...)'がうまく動作しない場合、あなたのサーバはおそらく動作していないか、正しく初期化されていません。 2番目のコードブロックは正しくあり、他のことが正しく行われていれば動作します。したがって、2番目のコードブロックが機能しない場合、コードには何か間違ったことがあります。他に何が間違っているかを知るためにコードの残りの部分を見なければなりません。 – jfriend00

答えて

3

エクスプレスミドルウェアについて知るべき重要なことの

var app = express(); 

// view engine setup 
app.set('views', path.join(__dirname, 'views')); 
app.set('view engine', 'hbs'); 

app.use(logger('dev')); 
app.use(bodyParser.json()); 
app.use(bodyParser.urlencoded({ extended: false })); 
app.use(cookieParser()); 
app.use(express.static(path.join(__dirname, 'public'))); 
app.use(session({ secret: 'separation anxiety' })); 
app.use(passport.initialize()); 
app.use(passport.session()); 

app.use('/', routes); 
app.use('/users', users); 

//たくさんのたくさんの宣言の順番で呼ばれているということです。

宣言されたミドルウェアのいずれかが要求を処理した場合(応答を返すことによって)、その要求を処理したミドルウェアには要求が渡されません。

他のすべてのミドルウェアが宣言された後に「キャッチオール」ミドルウェアを追加すると、他のどこかで処理されたため、ミドルウェアに要求が渡されることはありません。

これを修正するには、ミドルウェアをミドルウェアチェーンのどこかで「上に」置いてください。あなたのケースでは、express.static前にそれを宣言してみてください。

app.use(function(req, res, next){ 
    console.log("--------new middleware"); 
    next(); 
}); 
app.use(express.static(path.join(__dirname, 'public'))); 

これは、ミドルウェア・チェーン内の正しい場所がどうなるか、あなたのミドルウェアの究極の目標に依存します。

または、404ハンドラのすぐ上にミドルウェアを追加したため、サーバーに存在しないURLをリクエストしてみてください。あなたのミドルウェアも同様にトリガーするはずです。

+0

存在しないURLを要求すると、すべてのコールまたは少なくともすべてのルートでこれを実行する方法はありますか? – Mankind1023

+0

@ Mankind1023はい、ルートが宣言されている場所の上に移動します。 – robertklep

+0

これをファイルの先頭に置くことは、このトリックをやっているようですね、ありがとう! – Mankind1023

関連する問題