[Fix] lua_scanners - fix luacheck

diff --git a/lualib/lua_scanners/common.lua b/lualib/lua_scanners/common.lua
index 1fe82fabb5c1061dac3e6c14dccf0be9bf7868b2..43400c3753a9ac3ba45f141491968ceb3d07fa81 100644 (file)
--- a/lualib/lua_scanners/common.lua
+++ b/lualib/lua_scanners/common.lua
@@ -180,7 +180,7 @@ local function save_av_cache(task, digest, rule, to_save, dyn_weight)
   return false
 end
 
-local function create_regex_table(task, patterns)
+local function create_regex_table(patterns)
   local regex_table = {}
   if patterns[1] then
     for i, p in ipairs(patterns) do

diff --git a/lualib/lua_scanners/oletools.lua b/lualib/lua_scanners/oletools.lua
index fd2775775a83bc19146c99196a1bd95b39299a1a..814fe368e605a1a23b5ddc5b6a60da9b870e17e1 100644 (file)
--- a/lualib/lua_scanners/oletools.lua
+++ b/lualib/lua_scanners/oletools.lua
@@ -27,7 +27,6 @@ local upstream_list = require "rspamd_upstream_list"
 local rspamd_logger = require "rspamd_logger"
 local ucl = require "ucl"
 local common = require "lua_scanners/common"
-local fun = require "fun"
 
 local module_name = 'oletools'
 

diff --git a/lualib/lua_scanners/sophos.lua b/lualib/lua_scanners/sophos.lua
index 0f874e75638f4edc3cfc68d99bfc4a2cd20bce76..d0297fa97d313daba32cd411e31adbf65603ce1e 100644 (file)
--- a/lualib/lua_scanners/sophos.lua
+++ b/lualib/lua_scanners/sophos.lua
@@ -129,9 +129,9 @@ local function sophos_check(task, content, digest, rule)
         else
           if string.find(data, 'DONE OK') then
             if rule['log_clean'] then
-              rspamd_logger.infox(task, '%s [%s]: message or mime_part is clean', rule['symbol'], rule['type'])
+              rspamd_logger.infox(task, '%s: message or mime_part is clean', rule.log_prefix)
             else
-              lua_util.debugm(rule.module_name, task, '%s [%s]: message or mime_part is clean', rule['symbol'], rule['type'])
+              lua_util.debugm(rule.module_name, task, '%s: message or mime_part is clean', rule.log_prefix)
             end
             common.save_av_cache(task, digest, rule, 'OK')
             -- not finished - continue

diff --git a/src/plugins/lua/antivirus.lua b/src/plugins/lua/antivirus.lua
index 0515e3376f7c27aba56c37e379660a2124e13bc1..5ce6dfe06fbbfabda79abecee385ccdb853b1cf1 100644 (file)
--- a/src/plugins/lua/antivirus.lua
+++ b/src/plugins/lua/antivirus.lua
@@ -15,7 +15,6 @@ limitations under the License.
 ]] --
 
 local rspamd_logger = require "rspamd_logger"
-local rspamd_regexp = require "rspamd_regexp"
 local lua_util = require "lua_util"
 local fun = require "fun"
 local lua_antivirus = require("lua_scanners").filter('antivirus')
@@ -108,7 +107,7 @@ local function add_antivirus_rule(sym, opts)
     return nil
   end
 
-  rule.patterns = common.create_regex_table(task, opts.patterns or {})
+  rule.patterns = common.create_regex_table(opts.patterns or {})
 
   if opts['whitelist'] then
     rule['whitelist'] = rspamd_config:add_hash_map(opts['whitelist'])

diff --git a/src/plugins/lua/external_services.lua b/src/plugins/lua/external_services.lua
index 038142da528ee8a6ec83931304fa00e2ee77fcbe..fb7aef7094203ba7d3c275b84a59e1c6b0e1d37a 100644 (file)
--- a/src/plugins/lua/external_services.lua
+++ b/src/plugins/lua/external_services.lua
@@ -147,11 +147,11 @@ local function add_scanner_rule(sym, opts)
       rule.scan_all_mime_parts = false
   end
 
-  rule.patterns = common.create_regex_table(task, opts.patterns or {})
+  rule.patterns = common.create_regex_table(opts.patterns or {})
 
-  rule.mime_parts_filter_regex = common.create_regex_table(task, opts.mime_parts_filter_regex or {})
+  rule.mime_parts_filter_regex = common.create_regex_table(opts.mime_parts_filter_regex or {})
 
-  rule.mime_parts_filter_ext = common.create_regex_table(task, opts.mime_parts_filter_ext or {})
+  rule.mime_parts_filter_ext = common.create_regex_table(opts.mime_parts_filter_ext or {})
 
   if opts['whitelist'] then
     rule['whitelist'] = rspamd_config:add_hash_map(opts['whitelist'])