fix(rate limits): Duration was set to 1 second, not 1 minute. Fixed

This commit is contained in:
Chris Troutner
2020-02-28 08:05:16 -08:00
parent 8974d9f025
commit 18812e0473
2 changed files with 5 additions and 1 deletions
+1 -1
View File
@@ -26,7 +26,7 @@ const { RateLimiterRedis } = require('rate-limiter-flexible')
const rateLimitOptions = {
storeClient: redisClient,
points: 100, // Number of points
duration: 1 // Per second
duration: 60 // Per second
}
// This hard-coded value is temporary. It will be swapped out with an environment
+4
View File
@@ -94,6 +94,8 @@ describe('#route-ratelimits & jwt-auth', () => {
let routeRateLimit = rateLimitMiddleware.routeRateLimit
// const getInfo = controlRoute.testableComponents.getInfo
// NOTE: this test will fail if you run multiple integration tests in a
// short period. Because it talks to the Redis DB.
it('should pass through rate-limit middleware', async () => {
req.baseUrl = '/v3'
req.path = '/control/getNetworkInfo'
@@ -433,6 +435,8 @@ describe('#route-ratelimits & jwt-auth', () => {
})
describe('#rateLimitByResource', () => {
// NOTE: this test will fail if you run multiple integration tests in a
// short period. Because it talks to the Redis DB.
it('should pass through rate-limit middleware', async () => {
req.baseUrl = '/v3'
req.path = '/control/getNetworkInfo'