[Bug 7370] New: NUMERIC_HTTP_ADDR misfires on numeric hostnames

[bu...@bugzilla.spamassassin.org]

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

            Bug ID: 7370
           Summary: NUMERIC_HTTP_ADDR misfires on numeric hostnames
           Product: Spamassassin
           Version: 3.4.1
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: minor
          Priority: P2
         Component: Rules
          Assignee: dev@spamassassin.apache.org
          Reporter: kdeugau@vianet.ca
  Target Milestone: Undefined

I just received a false-positive report where NUMERIC_HTTP_ADDR fired on an
all-numeric hostname part of a longer fully-qualified domain name.

The domain part of the link was:

https://2987979.fls.doubleclick.net/

Here's a suggested fix to ensure that it is really matching on the entire
fully-qualified hostname part of the URI:

uri NUMERIC_HTTP_ADDR  m{^https?\://\d{7}(?:/|$)}is

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

[Bug 7370] NUMERIC_HTTP_ADDR misfires on numeric hostnames

[bu...@bugzilla.spamassassin.org]

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

Kevin A. McGrail <km...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kmcgrail@apache.org

--- Comment #1 from Kevin A. McGrail <km...@apache.org> ---
That fix won't work because I think he's really just looking for at least 7
numbers not exactly 7 numbers

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

[Bug 7370] NUMERIC_HTTP_ADDR misfires on numeric hostnames

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

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

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

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

--- Comment #2 from Henrik Krohns <he...@hege.li> ---
Who knows what the rules original intention was. Doesn't even hit anything
right now. Fixing it to match only numeric/ips.

Sending        rules/20_uri_tests.cf
Transmitting file data .done
Committing transaction...
Committed revision 1861579.

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