You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@spamassassin.apache.org on 2022/11/28 16:21:58 UTC
[Bug 8077] New: HashBL "number" options
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8077
Bug ID: 8077
Summary: HashBL "number" options
Product: Spamassassin
Version: unspecified
Hardware: PC
OS: Linux
Status: NEW
Severity: enhancement
Priority: P2
Component: Libraries
Assignee: dev@spamassassin.apache.org
Reporter: giovanni@paclan.it
Target Milestone: Undefined
Created attachment 5859
--> https://bz.apache.org/SpamAssassin/attachment.cgi?id=5859&action=edit
Num HashBL option
In some cases, chars matched by check_hashbl_bodyre() are not clean and
extraneous chars should be cleaned before sending the dns query.
When trying to match phone numbers, for example, the check_hashbl_bodyre()
match may be of the form "+1 (123) 456-7890", the attached diff will remove non
numbers chars from the match.
--
You are receiving this mail because:
You are the assignee for the bug.
[Bug 8077] HashBL "number" options
Posted by bu...@spamassassin.apache.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8077
Henrik Krohns <ap...@hege.li> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |apache@hege.li
--- Comment #1 from Henrik Krohns <ap...@hege.li> ---
I think if it's ok if we check that empty results don't create unnecessary
lookups (just in case someone does bad regexps).
push @matches, $match;
-->
push @matches, $match if $match ne '';
--
You are receiving this mail because:
You are the assignee for the bug.
[Bug 8077] HashBL "number" options
Posted by bu...@spamassassin.apache.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8077
Kevin A. McGrail <km...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kmcgrail@apache.org
Target Milestone|Undefined |4.0.0
--- Comment #3 from Kevin A. McGrail <km...@apache.org> ---
I'm +1 on this and it should be safe for 4.0. That gives us 3 +1s.
--
You are receiving this mail because:
You are the assignee for the bug.
[Bug 8077] HashBL "number" options
Posted by bu...@spamassassin.apache.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8077
Giovanni Bechis <gi...@paclan.it> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
CC| |giovanni@paclan.it
Resolution|--- |FIXED
--- Comment #5 from Giovanni Bechis <gi...@paclan.it> ---
Sending lib/Mail/SpamAssassin/Plugin/HashBL.pm
Transmitting file data .done
Committing transaction...
Committed revision 1905766.
--
You are receiving this mail because:
You are the assignee for the bug.
[Bug 8077] HashBL "number" options
Posted by bu...@spamassassin.apache.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8077
--- Comment #4 from Henrik Krohns <ap...@hege.li> ---
Yes +1 for fast commit so we can get on with 4.0.0
--
You are receiving this mail because:
You are the assignee for the bug.
[Bug 8077] HashBL "number" options
Posted by bu...@spamassassin.apache.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8077
Giovanni Bechis <gi...@paclan.it> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #5859|0 |1
is obsolete| |
--- Comment #2 from Giovanni Bechis <gi...@paclan.it> ---
Created attachment 5860
--> https://bz.apache.org/SpamAssassin/attachment.cgi?id=5860&action=edit
Num HashBL option
Updated patch with hege@ suggestions.
--
You are receiving this mail because:
You are the assignee for the bug.