From 3bca856e87467bb2f50b59a903b2302e313b7f7d Mon Sep 17 00:00:00 2001 From: Jon Chambers Date: Tue, 7 Jul 2020 15:32:50 -0400 Subject: [PATCH] Remove a pair of spurious SET calls in the rate limiter script. --- service/src/main/resources/lua/validate_rate_limit.lua | 3 --- 1 file changed, 3 deletions(-) diff --git a/service/src/main/resources/lua/validate_rate_limit.lua b/service/src/main/resources/lua/validate_rate_limit.lua index 17c1e2215..442e8c330 100644 --- a/service/src/main/resources/lua/validate_rate_limit.lua +++ b/service/src/main/resources/lua/validate_rate_limit.lua @@ -21,9 +21,6 @@ end local elapsedTime = currentTimeMillis - leakyBucket["lastUpdateTimeMillis"] local updatedSpaceRemaining = math.min(leakyBucket["bucketSize"], math.floor(leakyBucket["spaceRemaining"] + (elapsedTime * leakyBucket["leakRatePerMillis"]))) -redis.call("SET", "elapsedTime", elapsedTime) -redis.call("SET", "updatedSpaceRemaining", updatedSpaceRemaining) - if updatedSpaceRemaining >= amount then leakyBucket["spaceRemaining"] = updatedSpaceRemaining - amount redis.call("SET", bucketId, cjson.encode(leakyBucket))