From: Vsevolod Stakhov <vsevolod@highsecure.ru>
Date: Fri, 16 Aug 2019 17:34:47 +0000 (+0100)
Subject: [Minor] External_services: Fix symbol fail assignment
X-Git-Tag: 2.0~401
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=61512ce9488c18d2daf56f17efffa4287317cc5a;p=rspamd.git

[Minor] External_services: Fix symbol fail assignment
---

diff --git a/src/plugins/lua/external_services.lua b/src/plugins/lua/external_services.lua
index 9aba7f8a6..7b20e629c 100644
--- a/src/plugins/lua/external_services.lua
+++ b/src/plugins/lua/external_services.lua
@@ -126,14 +126,7 @@ local function add_scanner_rule(sym, opts)
     return nil
   end
 
-  if not opts.symbol_fail then
-    opts.symbol_fail = opts.symbol .. '_FAIL'
-  end
-
   local rule = cfg.configure(opts)
-  rule.type = opts.type
-  rule.symbol_fail = opts.symbol_fail
-  rule.redis_params = redis_params
 
   if not rule then
     rspamd_logger.errx(rspamd_config, 'cannot configure %s for %s',
@@ -141,6 +134,14 @@ local function add_scanner_rule(sym, opts)
     return nil
   end
 
+  rule.type = opts.type
+
+  if not rule.symbol_fail then
+    rule.symbol_fail = opts.symbol .. '_FAIL'
+  end
+
+  rule.redis_params = redis_params
+
   -- if any mime_part filter defined, do not scan all attachments
   if opts.mime_parts_filter_regex ~= nil
     or opts.mime_parts_filter_ext ~= nil then