From: Steve Freegard <steve@stevefreegard.com>
Date: Mon, 19 Mar 2018 11:53:38 +0000 (+0000)
Subject: Split MISSING_SUBJECT into EMPTY_SUBJECT based on corpus testing
X-Git-Tag: 1.7.1~6^2~7
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=8102f9c77e04fdbf63619f8e9d90c263078d5bcc;p=rspamd.git

Split MISSING_SUBJECT into EMPTY_SUBJECT based on corpus testing
---

diff --git a/rules/regexp/headers.lua b/rules/regexp/headers.lua
index e80380197..2570e9f75 100644
--- a/rules/regexp/headers.lua
+++ b/rules/regexp/headers.lua
@@ -62,17 +62,23 @@ reconf['R_NO_SPACE_IN_FROM'] = {
   group = 'header'
 }
 
-rspamd_config.MISSING_SUBJECT = {
+-- Detects missing Subject header
+reconf['MISSING_SUBJECT'] = {
+  re = '!raw_header_exists(Subject)',
   score = 2.0,
-  description = 'Subject is missing inside message',
+  description = 'Subject header is missing',
+  group = 'header'
+}
+
+rspamd_config.EMPTY_SUBJECT = {
+  score = 1.0,
+  description = 'Subject header is empty',
   group = 'header',
   callback = function(task)
     local hdr = task:get_header('Subject')
-
-    if not hdr or #hdr == 0 then
+    if hdr and #hdr == 0 then
       return true
     end
-
     return false
   end
 }