Merge branch 'master' into assets-route

This commit is contained in:
Peter Jaszkowiak
2017-02-06 15:52:54 -07:00
committed by GitHub
148 changed files with 1649 additions and 1357 deletions

View File

@@ -12,33 +12,7 @@ module.exports = function (middleware) {
}
var url = req.url.replace(nconf.get('relative_path'), '');
var allowedRoutes = [
'^/ping',
'^/sping',
'^/login',
'^/stylesheet.css',
'^/favicon',
'^/nodebb.min.js',
'^/vendor/fontawesome/fonts/fontawesome-webfont.woff',
'^/src/(modules|client)/[\\w/]+.js',
'^/templates/[\\w/]+.tpl',
'^/api/login',
'^/api/widgets/render',
'^/public/language',
'^/uploads/system/site-logo.png'
];
var isAllowed = function (url) {
for(var x = 0,numAllowed = allowedRoutes.length,route; x < numAllowed; x++) {
route = new RegExp(allowedRoutes[x]);
if (route.test(url)) {
return true;
}
}
return false;
};
if (isAllowed(url)) {
if (url.startsWith('/login') || url.startsWith('/api/login')) {
return next();
}

View File

@@ -3,7 +3,7 @@
'use strict';
var winston = require('winston');
var ratelimit = {};
var ratelimit = module.exports;
var allowedCalls = 100;
var timeframe = 10000;
@@ -31,5 +31,3 @@ ratelimit.isFlooding = function (socket) {
socket.lastCallTime = now;
return false;
};
module.exports = ratelimit;