https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6549

--- Comment #5 from Adam Katz <[email protected]> 2011-12-10 19:36:47 UTC ---
My only hesitation with that code is the   /(${IP_ADDRESS}).{10,80}/   part,
where ".{10,80}" could conceivably continue the IP given the lack of a
delimiter.  The IP_ADDRESS constant ends with   (?![a-f0-9:])   but lacks a
IPv4-ish equivalent, thus allowing mathces like "1.2.3.4.example.com" or even
"1.2.3.4.5"

This regex is the only one that contains an ambiguous ending to an IP matcher,
at least that I could find with   /IP_ADDRESS\}?\)?[\[.]/


This really should be a change to IP_ADDRESS itself, but for consistency, I'd
like to change the regex here to include a space after ${IP_ADDRESS}

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to