エクスプレス：私はhttpをhttpsにリダイレクトする簡単なスクリプトを持っているのparam

Question

var express = require('express'); 
var app = express(); 

app.get('*', function (req, res) { 
    var host = req.get('Host'); 
    return res.redirect(['https://', host, req.url].join('')); 
}); 

var server = app.listen(8080, function() { 
    console.log('starting'); 
}); 

をデコードするのに失敗しました。悪質なURL（例：website.com/%c0%ae%c0%ae）がある場合を除き、これは正常に動作します。

URIError: Failed to decode param '/%c0%ae%c0%ae' 
    at decodeURIComponent (native) 
    at decode_param (/...<PROJECT DIRECTORY>.../node_modules/express/lib/router/layer.js:167:12) 
    at Layer.match (/.../node_modules/express/lib/router/layer.js:143:15) 
    at matchLayer (/.../node_modules/express/lib/router/index.js:557:18) 
    at next (/.../node_modules/express/lib/router/index.js:216:15) 
    at expressInit (/.../node_modules/express/lib/middleware/init.js:33:5) 
    at Layer.handle [as handle_request] (/.../node_modules/express/lib/router/layer.js:95:5) 
    at trim_prefix (/.../node_modules/express/lib/router/index.js:312:13) 
    at /.../node_modules/express/lib/router/index.js:280:7 
    at Function.process_params (/.../node_modules/express/lib/router/index.js:330:12) 

ユーザーが自分のプロジェクトファイルがサーバー内のどこにあるかをランダムに見ることができたらうれしいことではありません。このエラーを処理する方法はありますか？

Answer 1

おかげ@Oleg。しかし、どういうわけかあなたの解決策は私のためにエラーを記録していなかった。ここに私が思いついたことがあります：

var express = require('express'); 
var app = express(); 

app.use(function(req, res, next) { 
    var err = null; 
    try { 
     decodeURIComponent(req.path) 
    } 
    catch(e) { 
     err = e; 
    } 
    if (err){ 
     console.log(err, req.url); 
     return res.redirect(['https://', req.get('Host'), '/404'].join(''));  
    } 
    next(); 
}); 

app.get('*', function (req, res) { 
    return res.redirect(['https://', req.get('Host'), req.url].join('')); 
}); 

var server = app.listen(8080, function() { 
    console.log('Starting'); 
}); 

Answer 2

可能な回避策：先端のための

var express = require('express'); 
var app = express(); 

app.get('*', function (req, res) { 
    // redirect regular paths 
    var host = req.get('Host'); 
    return res.redirect(['https://', host, req.url].join('')); 
}); 

// your express error handler 
app.use(function(err, req, res, next) { 
    // in case of specific URIError 
    if (err instanceof URIError) { 
     err.message = 'Failed to decode param: ' + req.url; 
     err.status = err.statusCode = 400; 

     // .. your redirect here if still needed 
     return res.redirect(['https://', req.get('Host'), req.url].join('')); 
    } else { 
     // .. 
    } 
    // .. 
}); 

var server = app.listen(8080, function() { 
    console.log('starting'); 
});