diff options
| author | Andrew Lewis <nerf@judo.za.org> | 2017-03-24 13:40:09 +0200 |
|---|---|---|
| committer | Andrew Lewis <nerf@judo.za.org> | 2017-03-24 13:40:09 +0200 |
| commit | ec68fae35b8cb4a4d97b2a5acb056472456db887 (patch) | |
| tree | 54606b0e93837bd1562ea10c20752d5196063a80 /rules/misc.lua | |
| parent | 573b224dc056371c0fbead54eb92101b81737a11 (diff) | |
| download | rspamd-ec68fae35b8cb4a4d97b2a5acb056472456db887.tar.gz rspamd-ec68fae35b8cb4a4d97b2a5acb056472456db887.zip | |
[Minor] Fix tests
Diffstat (limited to 'rules/misc.lua')
| -rw-r--r-- | rules/misc.lua | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/rules/misc.lua b/rules/misc.lua index 63c52eb4f..9ac5c1803 100644 --- a/rules/misc.lua +++ b/rules/misc.lua @@ -401,7 +401,6 @@ rspamd_config:register_symbol{ } local check_from_display_name = rspamd_config:register_symbol{ - name = 'CHECK_FROM_SPOOF', callback = function (task) local from = task:get_from(2) if not (from and from[1] and from[1].name) then return false end @@ -414,7 +413,7 @@ local check_from_display_name = rspamd_config:register_symbol{ if not util.strequal_caseless(from[1]['domain'], parsed[1]['domain']) then -- See if the destination domain is the same as the spoof local to = task:get_recipients(2) - if not (to and to[1] and to[1]['domain']) then + if not (to and to[1] and to[1]['domain']) then -- Be careful with undisclosed-recipients:; as domain will be an empty string if to[1]['domain'] == nil or to[1]['domain'] == '' then return false end task:insert_result('FROM_NEQ_DISPLAY_NAME', 1.0, from[1]['domain'], parsed[1]['domain']) @@ -455,13 +454,13 @@ rspamd_config.SPOOF_REPLYTO = { local to = task:get_recipients(2) if not (from and from[1] and from[1].addr) then return false end if (to and to[1] and to[1].addr) then - -- Handle common case for Web Contact forms of From = To + -- Handle common case for Web Contact forms of From = To if util.strequal_caseless(from[1].addr, to[1].addr) then return false end - end + end -- SMTP recipients must contain From domain - local to = task:get_recipients(1) + to = task:get_recipients(1) if not to then return false end local found_fromdom = false for _, t in ipairs(to) do |