From: Vsevolod Stakhov <vsevolod@rspamd.com>
Date: Sun, 22 May 2022 11:55:42 +0000 (+0100)
Subject: [Minor] History_redis: Allow no expire as a valid expire value
X-Git-Tag: 3.3~226
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=ce6dd7ac67d25be0b07a1a8e14ca2b4d04c6b70f;p=thirdparty%2Frspamd.git

[Minor] History_redis: Allow no expire as a valid expire value
---

diff --git a/src/plugins/lua/history_redis.lua b/src/plugins/lua/history_redis.lua
index 9f8196c1c0..b8bd9466a6 100644
--- a/src/plugins/lua/history_redis.lua
+++ b/src/plugins/lua/history_redis.lua
@@ -55,7 +55,7 @@ local redis_params
 
 local settings = {
   key_prefix = 'rs_history', -- default key name
-  expire = 0, -- default no expire
+  expire = nil, -- default no expire
   nrows = 200, -- default rows limit
   compress = true, -- use zstd compression when storing data in redis
   subject_privacy = false, -- subject privacy is off
@@ -169,7 +169,7 @@ local function history_save(task)
   if ret then
     conn:add_cmd('LTRIM', {prefix, '0', string.format('%d', settings.nrows-1)})
 
-    if settings.expire > 0 then
+    if settings.expire and settings.expire > 0 then
       conn:add_cmd('EXPIRE', {prefix, string.format('%d', settings.expire)})
     end
   end