]> git.ipfire.org Git - thirdparty/rspamd.git/commitdiff
[Minor] Include module name when setting pre result
authorVsevolod Stakhov <vsevolod@highsecure.ru>
Sat, 13 Oct 2018 12:30:07 +0000 (13:30 +0100)
committerVsevolod Stakhov <vsevolod@highsecure.ru>
Sat, 13 Oct 2018 12:30:23 +0000 (13:30 +0100)
src/plugins/lua/antivirus.lua
src/plugins/lua/dmarc.lua
src/plugins/lua/force_actions.lua
src/plugins/lua/greylist.lua
src/plugins/lua/metadata_exporter.lua
src/plugins/lua/multimap.lua
src/plugins/lua/ratelimit.lua
src/plugins/lua/replies.lua
src/plugins/lua/spamtrap.lua

index 91a6c0a9b1b614c1e994d505ef63404f1b9d105a..025e3604323b8121bd57b9fc2c7a301e77409cc5 100644 (file)
@@ -123,7 +123,7 @@ local function yield_result(task, rule, vname)
         lua_util.template(rule.message or 'Rejected', {
           SCANNER = rule['type'],
           VIRUS = vname,
-        }))
+        }), N)
   end
 end
 
index e632c9f02454e306c8e58bdbf8cb72774c2b0ae3..b55c5d41f68e3945b1bb4f730c22b1915e7c7976 100644 (file)
@@ -201,7 +201,7 @@ local function maybe_force_action(task, disposition)
     if force_action then
       -- Don't do anything if pre-result has been already set
       if task:has_pre_result() then return end
-      task:set_pre_result(force_action, 'Action set by DMARC')
+      task:set_pre_result(force_action, 'Action set by DMARC', N)
     end
   end
 end
index 6c898b8ff87dc8157092d54249977f358a327499..eddd09deac531c4232629064a6ade4483809fc79 100644 (file)
@@ -79,9 +79,9 @@ local function gen_cb(expr, act, pool, message, subject, raction, honor, limit)
         task:set_metric_subject(subject)
       end
       if type(message) == 'string' then
-        task:set_pre_result(act, message)
+        task:set_pre_result(act, message, N)
       else
-        task:set_pre_result(act)
+        task:set_pre_result(act, nil, N)
       end
       return true, act
     end
index 0bf74bc74184375b6c5c37fe1df6058053f449ee..7427f999e42847faefc594b4872016314c1b36a1 100644 (file)
@@ -157,13 +157,13 @@ local function greylist_message(task, end_time, why)
   if rspamd_lua_utils.is_rspamc_or_controller(task) then return end
   if settings.message_func then
     task:set_pre_result(settings['action'],
-      settings.message_func(task, end_time))
+      settings.message_func(task, end_time), N)
   else
     local message = settings['message']
     if settings.report_time then
       message = string.format("%s: %s", message, end_time)
     end
-    task:set_pre_result(settings['action'], message)
+    task:set_pre_result(settings['action'], message, N)
   end
 
   task:set_flag('greylisted')
index 6c0eb6f825112eb789426dc2614a06db1dbbe0a8..57fe0f105e38ccafbdb1223f463ae19ff6415476 100644 (file)
@@ -229,7 +229,7 @@ local selectors = {
 local function maybe_defer(task, rule)
   if rule.defer then
     rspamd_logger.warnx(task, 'deferring message')
-    task:set_pre_result('soft reject', 'deferred')
+    task:set_pre_result('soft reject', 'deferred', N)
   end
 end
 
index 6a12d898a5b884ca332d7d2e3a3361e9f49d8bbf..4cc9af48b05d6871211590850dbcb65e47abf446 100644 (file)
@@ -555,9 +555,9 @@ local function multimap_callback(task, rule)
             r['message'] = r.message_func(task, r['symbol'], opt)
           end
           if r['message'] then
-            task:set_pre_result(r['action'], r['message'])
+            task:set_pre_result(r['action'], r['message'], N)
           else
-            task:set_pre_result(r['action'], 'Matched map: ' .. r['symbol'])
+            task:set_pre_result(r['action'], 'Matched map: ' .. r['symbol'], N)
           end
         end
       end
@@ -731,7 +731,7 @@ local function multimap_callback(task, rule)
           elseif is_ok then
             task:insert_result(rule['symbol'], 1, rule['map'])
             if pre_filter then
-              task:set_pre_result(rule['action'], 'Matched map: ' .. rule['symbol'])
+              task:set_pre_result(rule['action'], 'Matched map: ' .. rule['symbol'], N)
             end
           end
 
index 05b1f5ed8a26b4977f072e67938b2f96f31979a1..61d19966bf5aeec5640505d2fe8f2baf5d59deb8 100644 (file)
@@ -556,7 +556,7 @@ local function ratelimit_cb(task)
               bucket.burst, bucket.rate,
               data[2], data[3], data[4])
           task:set_pre_result('soft reject',
-              message_func(task, lim_name, prefix, bucket))
+              message_func(task, lim_name, prefix, bucket), N)
         end
       end
     end
index e16ef2b7d3444c173b70398a3b0c88952c3d0f59..fe15211efe506ecf7d8d13fd6f7094c0d3099d27 100644 (file)
@@ -65,7 +65,7 @@ local function replies_check(task)
             (settings.use_local and ip_addr and ip_addr:is_local()) then
           rspamd_logger.infox(task, "not forcing action for local network or authorized user");
         else
-          task:set_pre_result(settings['action'], settings['message'])
+          task:set_pre_result(settings['action'], settings['message'], N)
         end
       end
     end
index 85af46dd238cb62a052feb3e2c0473ee65f153d5..97107c2756f57d3cdf78c276da64720864ba3164 100644 (file)
@@ -66,7 +66,7 @@ local function spamtrap_cb(task)
       rspamd_logger.infox(task, 'spamtrap found: <%s>', rcpt)
       if settings.smtp_message then
         task:set_pre_result(settings['action'],
-          lua_util.template(settings.smtp_message, { rcpt = rcpt}))
+          lua_util.template(settings.smtp_message, { rcpt = rcpt}), 'spamtrap')
       else
         local smtp_message = 'unknown error'
         if settings.action == 'no action' then
@@ -74,7 +74,7 @@ local function spamtrap_cb(task)
         elseif settings.action == 'reject' then
           smtp_message = 'message rejected'
         end
-        task:set_pre_result(settings['action'], smtp_message)
+        task:set_pre_result(settings['action'], smtp_message, 'spamtrap')
       end
     end
   end