Skip to content

Commit 3e2c6d6

Browse files
authored
Merge pull request github#13940 from erik-krogh/rate-default
JS: change the defaults in the qhelp for missing-rate-limit to something more reasonable
2 parents 5a6ce29 + 5ffce86 commit 3e2c6d6

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

javascript/ql/src/Security/CWE-770/examples/MissingRateLimitingGood.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ var app = express();
44
// set up rate limiter: maximum of five requests per minute
55
var RateLimit = require('express-rate-limit');
66
var limiter = RateLimit({
7-
windowMs: 1*60*1000, // 1 minute
8-
max: 5
7+
windowMs: 15 * 60 * 1000, // 15 minutes
8+
max: 100, // max 100 requests per windowMs
99
});
1010

1111
// apply rate limiter to all requests

0 commit comments

Comments
 (0)