From: Vsevolod Stakhov <vsevolod@highsecure.ru>
Date: Tue, 11 Jan 2022 20:25:52 +0000 (+0000)
Subject: [Minor] Fix some issues in URI_COUNT_ODD rule
X-Git-Tag: 3.2~106
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c6f7b897d45cbe4a166334ca60d29b3b7391bd3a;p=thirdparty%2Frspamd.git

[Minor] Fix some issues in URI_COUNT_ODD rule

Issue: #4037
---

diff --git a/rules/misc.lua b/rules/misc.lua
index 2e5f388007..9c8cade710 100644
--- a/rules/misc.lua
+++ b/rules/misc.lua
@@ -338,10 +338,8 @@ rspamd_config.URI_COUNT_ODD = {
   callback = function (task)
     local ct = task:get_header('Content-Type')
     if (ct and ct:lower():find('^multipart/alternative')) then
-      local urls = task:get_urls() or {}
-      local nurls = fun.filter(function(url)
-        return not url:is_html_displayed()
-      end, urls):foldl(function(acc, val) return acc + val:get_count() end, 0)
+      local urls = task:get_urls_filtered(nil, {'subject', 'html_displayed', 'special'}) or {}
+      local nurls = fun.foldl(function(acc, val) return acc + val:get_count() end, 0)
 
       if nurls % 2 == 1 then
         return true, 1.0, tostring(nurls)