Fix monitoring being blocked

This commit is contained in:
Gaël Métais
2021-03-11 09:56:41 +00:00
parent 92d92b8021
commit fcdf3c9240
2 changed files with 9 additions and 1 deletions
+1 -1
View File
@@ -538,7 +538,7 @@ var policies = {
"message": "<p>It can be useful, but only as a last resort. It is a bad practice because it overrides the normal cascading logic. The more you use !important, the more you need it again to over-override. This conducts to a poor maintainability.</p>",
"isOkThreshold": 0,
"isBadThreshold": 200,
"isAbnormalThreshold": 500,
"isAbnormalThreshold": 1000,
"hasOffenders": true,
"offendersTransformFn": function(offenders) {
var parsedOffenders = offenders.map(function(offender) {
@@ -13,6 +13,13 @@ var apiLimitsMiddleware = function(req, res, next) {
if (req.path.indexOf('/api/') === 0 && !res.locals.hasApiKey) {
// Monitoring requests
if (req.path === '/api/runs' && req.method === 'GET') {
next();
return;
}
// New tests
if (req.path === '/api/runs' && req.method === 'POST') {
if (!runsTable.accepts(ipAddress)) {
@@ -24,6 +31,7 @@ var apiLimitsMiddleware = function(req, res, next) {
}
// Every other calls
if (!callsTable.accepts(ipAddress)) {
// Sorry :/
debug('Too many API requests from IP address %s', ipAddress);