diff options
author | BoHong Li | 2017-04-12 05:30:04 +0800 |
---|---|---|
committer | Raccoon Li | 2017-05-08 19:24:37 +0800 |
commit | dee77c459a929d7e0041fae38176b71248a9ff9c (patch) | |
tree | 1c0e669e6ffe1bd032738915aec1756ba959d85f /app.js | |
parent | 7ba0d600f1ae5f2e7cf8bdc51ae32dc6b823d781 (diff) |
refactor(app.js): Extract middleware to module
extract check URi is valid, redirect without trailing slashes
Diffstat (limited to '')
-rw-r--r-- | app.js | 22 |
1 files changed, 2 insertions, 20 deletions
@@ -192,28 +192,10 @@ passport.deserializeUser(function (id, done) { }) // check uri is valid before going further -app.use(function (req, res, next) { - try { - decodeURIComponent(req.path) - } catch (err) { - logger.error(err) - return response.errorBadRequest(res) - } - next() -}) +app.use(require('./lib/web/middleware/checkURiValid')) // redirect url without trailing slashes -app.use(function (req, res, next) { - if (req.method === 'GET' && req.path.substr(-1) === '/' && req.path.length > 1) { - var query = req.url.slice(req.path.length) - var urlpath = req.path.slice(0, -1) - var serverurl = config.serverurl - if (config.urlpath) serverurl = serverurl.slice(0, -(config.urlpath.length + 1)) - res.redirect(301, serverurl + urlpath + query) - } else { - next() - } -}) +app.use(require('./lib/web/middleware/redirectwithoutTrailingSlashes')) // routes need sessions // template files |