[Bug 7783] New: check_rbl and rule redefinition

[bu...@spamassassin.apache.org]

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7783

            Bug ID: 7783
           Summary: check_rbl and rule redefinition
           Product: Spamassassin
           Version: 3.4.3
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Rules
          Assignee: dev@spamassassin.apache.org
          Reporter: riccardo.alfieri@spamteq.com
  Target Milestone: Undefined

Hello,

I have two different SA installations, 3.4.2 and 3.4.3 . Both have this rule
redefinition:

header  __RCVD_IN_ZEN  eval:check_rbl('zen','<key>.zen.dq.spamhaus.net.')
header  __RCVD_IN_ZEN_LASTEXTERNAL      eval:check_rbl('zen-lastexternal',
'<key>.zen.dq.spamhaus.net.', '^127\.0\.0\.([2-9]|10|11)$')

What happens is that in 3.4.3 the rule __RCVD_IN_ZEN still uses
zen.spamhaus.org despite being redefined, while in 3.4.2 it works as expected.

I found this workaround for 3.4.3:

header  __RCVD_IN_ZEN  eval:check_rbl('zendqs','<key>.zen.dq.spamhaus.net.')
header  __RCVD_IN_ZEN_LASTEXTERNAL      eval:check_rbl('zendqs-lastexternal',
'<key>.zen.dq.spamhaus.net.', '^127\.0\.0\.([2-9]|10|11)$')

that is, using another name for the 'set' parameter in check_rbl. Doing this
the rule __RCVD_IN_ZEN is being correctly redefined and does the lookups only
on the .dq.spamhaus.net zone. 

Looks like if that I use the same 'set' name as a rule loaded before my custom
redefinition, that earlier 'set' takes precedence.

Is that an expected behaviour or am I doing something fundamentally wrong?

Thanks

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 7783] check_rbl and rule redefinition

[bu...@spamassassin.apache.org]

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7783

Henrik Krohns <ap...@hege.li> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |apache@hege.li
         Resolution|---                         |INVALID

--- Comment #1 from Henrik Krohns <ap...@hege.li> ---

Version should make no difference.

You need to redefine _all_ rules that refer to 'zen' set and zen.spamhaus.org.

Works fine for me:

header __RCVD_IN_ZEN      eval:check_rbl('zen', '<key>.zen.dq.spamhaus.net.')
header RCVD_IN_XBL  eval:check_rbl('zen-lastexternal',
'<key>.zen.dq.spamhaus.net.', '^127\.0\.0\.[4567]$')
header RCVD_IN_PBL  eval:check_rbl('zen-lastexternal',
'<key>.zen.dq.spamhaus.net.', '^127\.0\.0\.1[01]$')
header RCVD_IN_ZEN_BLOCKED_OPENDNS  eval:check_rbl('zen-lastexternal',
'<key>.zen.dq.spamhaus.net.', '^127\.255\.255\.254$')
header RCVD_IN_ZEN_BLOCKED          eval:check_rbl('zen-lastexternal',
'<key>.zen.dq.spamhaus.net.', '^127\.255\.255\.255$')

-- 
You are receiving this mail because:
You are the assignee for the bug.