From: Vsevolod Stakhov <vsevolod@rspamd.com>
Date: Thu, 29 Aug 2024 14:45:10 +0000 (+0100)
Subject: [Fix] Preserve the previous behaviour of RDNS_* checks
X-Git-Tag: 3.10.0~30^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=fb774ae9c5b0b4cb6b223764d9702630a1f70052;p=rspamd.git

[Fix] Preserve the previous behaviour of RDNS_* checks
---

diff --git a/rules/misc.lua b/rules/misc.lua
index 8785ab4ac..4ddb00dfb 100644
--- a/rules/misc.lua
+++ b/rules/misc.lua
@@ -863,6 +863,9 @@ rspamd_config.COMPLETELY_EMPTY = {
   score = 15
 }
 
+-- Preserve compatibility
+local rdns_auth_and_local_conf = lua_util.config_check_local_or_authed(rspamd_config, 'once_received',
+    false, false)
 -- Check for the hostname if it was not set
 local rnds_check_id = rspamd_config:register_symbol {
   name = 'RDNS_CHECK',
@@ -899,6 +902,15 @@ local rnds_check_id = rspamd_config:register_symbol {
   -- TODO: settings might need to use this symbol if they depend on hostname...
   priority = lua_util.symbols_priorities.top - 1,
   description = 'Check if hostname has been resolved by MTA',
+  condition = function(task)
+    local task_ip = task:get_ip()
+    if ((not rdns_auth_and_local_conf[1] and task:get_user()) or
+        (not rdns_auth_and_local_conf[2] and task_ip and task_ip:is_local())) then
+      return false
+    end
+
+    return true
+  end
 }
 
 rspamd_config:register_symbol {