]> source.dussan.org Git - rspamd.git/commitdiff
[Fix] rbl: fix `exclude_local` 4678/head
authorAndrew Lewis <nerf@judo.za.org>
Fri, 3 Nov 2023 15:26:53 +0000 (17:26 +0200)
committerAndrew Lewis <nerf@judo.za.org>
Fri, 3 Nov 2023 15:26:53 +0000 (17:26 +0200)
lualib/plugins/rbl.lua
src/plugins/lua/rbl.lua

index 320fd77251b855c934f7fb70db6df66b8a3564a4..bf8bd3cfac0777755530f9fb7bd504341a5ee0ca 100644 (file)
@@ -49,7 +49,6 @@ local default_options = {
   ['default_unknown'] = false,
   ['default_dkim_domainonly'] = true,
   ['default_emails_domainonly'] = false,
-  ['default_exclude_private_ips'] = true,
   ['default_exclude_users'] = false,
   ['default_exclude_local'] = true,
   ['default_no_ip'] = false,
@@ -91,7 +90,6 @@ local rule_schema_tbl = {
   emails_domainonly = ts.boolean:is_optional(),
   enabled = ts.boolean:is_optional(),
   exclude_local = ts.boolean:is_optional(),
-  exclude_private_ips = ts.boolean:is_optional(),
   exclude_users = ts.boolean:is_optional(),
   from = ts.boolean:is_optional(),
   hash = ts.one_of { "sha1", "sha256", "sha384", "sha512", "md5", "blake2" }:is_optional(),
index f2233a3e593c1f9ec2bdb5d76565b0c14497ffb9..fda7cce16edb466d604427a3c95f8f5168009f52 100644 (file)
@@ -151,9 +151,7 @@ local function gen_check_rcvd_conditions(rbl, received_total)
     end
     if ((rh.real_ip:get_version() == 6 and rbl.ipv6) or
         (rh.real_ip:get_version() == 4 and rbl.ipv4)) and
-        ((rbl.exclude_private_ips and not rh.real_ip:is_local()) or
-            not rbl.exclude_private_ips) and ((rbl.exclude_local_ips and
-        not is_excluded_ip(rh.real_ip)) or not rbl.exclude_local_ips) then
+        ((rbl.exclude_local and not rh.real_ip:is_local() or is_excluded_ip(rh.real_ip)) or not rbl.exclude_local) then
       return true
     else
       return false
@@ -845,7 +843,7 @@ local function gen_rbl_callback(rule)
     description[#description + 1] = 'user'
   end
 
-  if rule.exclude_local or rule.exclude_private_ips then
+  if rule.exclude_local then
     pipeline[#pipeline + 1] = check_local
     description[#description + 1] = 'local'
   end