fix: avoid double-dip of nrcpt when calculating ratelimit condition

Before this commit, rspamd would consider the number of recipients of
the current message twice when calculating the ratelimit condition.

diff --git a/lualib/redis_scripts/ratelimit_check.lua b/lualib/redis_scripts/ratelimit_check.lua
index aa7f564ef2fb6a9c0a37ef4e8e09d8e455c817fd..1c2b32a69222b561e57937142744f1ac38766a3b 100644 (file)
--- a/lualib/redis_scripts/ratelimit_check.lua
+++ b/lualib/redis_scripts/ratelimit_check.lua
@@ -62,7 +62,7 @@ if burst + pending > 0 then
   if dynb == 0 then dynb = 0.0001 end
 
   burst = burst + pending
-  if burst > 0 and (burst + nrcpt) > max_burst * dynb then
+  if burst > 0 and burst > max_burst * dynb then
     return {1, tostring(burst - pending), tostring(dynr), tostring(dynb), tostring(leaked)}
   end
   -- Increase pending if we allow ratelimit