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...@bugzilla.spamassassin.org on 2015/04/08 17:02:25 UTC

[Bug 7167] New: NAME_EMAIL_DIFF should not be case-sensitive

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

            Bug ID: 7167
           Summary: NAME_EMAIL_DIFF should not be case-sensitive
           Product: Spamassassin
           Version: unspecified
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: trivial
          Priority: P2
         Component: Rules
          Assignee: dev@spamassassin.apache.org
          Reporter: kdeugau@vianet.ca

I just received an FP report with the following From: header:

From: "Nattawuth@totalflow.co.th" <na...@totalflow.co.th>

It triggered NAME_EMAIL_DIFF even though the addresses are identical except for
case.

I suggest changing the subrule __NAME_EQ_EMAIL to be case-insensitive.

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

[Bug 7167] NAME_EMAIL_DIFF should not be case-sensitive

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7167

--- Comment #1 from Mark Martinec <Ma...@ijs.si> ---
> I suggest changing the subrule __NAME_EQ_EMAIL to be case-insensitive.


--- rulesrc/sandbox/khopesh/20_khop_experimental.cf~       2015-04-08
20:26:46.479635000 +0200
+++ rulesrc/sandbox/khopesh/20_khop_experimental.cf     2015-04-08
20:27:03.319635475 +0200
@@ -147,3 +147,3 @@
 header __NAME_IS_EMAIL From:raw =~ /\w\@[\w.-]+\.\w\w+["'`]*\s*<\w+\@\w/
-header __NAME_EQ_EMAIL From:raw =~ /([\w+.-]+\@[\w.-]+\.\w\w+)["'`\s]*<\s*\1>/
+header __NAME_EQ_EMAIL From:raw =~
/([\w+.-]+\@[\w.-]+\.\w\w+)["'`\s]*<\s*\1>/i
 meta    NAME_EMAIL_DIFF        __NAME_IS_EMAIL && ! __NAME_EQ_EMAIL


(and rules/72_active.cf)

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

[Bug 7167] NAME_EMAIL_DIFF should not be case-sensitive

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7167

Kevin A. McGrail <km...@pccc.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
                 CC|                            |kmcgrail@pccc.com

--- Comment #2 from Kevin A. McGrail <km...@pccc.com> ---
(In reply to Mark Martinec from comment #1)
> > I suggest changing the subrule __NAME_EQ_EMAIL to be case-insensitive.
> 
> 
> --- rulesrc/sandbox/khopesh/20_khop_experimental.cf~       2015-04-08
> 20:26:46.479635000 +0200
> +++ rulesrc/sandbox/khopesh/20_khop_experimental.cf     2015-04-08
> 20:27:03.319635475 +0200
> @@ -147,3 +147,3 @@
>  header __NAME_IS_EMAIL From:raw =~ /\w\@[\w.-]+\.\w\w+["'`]*\s*<\w+\@\w/
> -header __NAME_EQ_EMAIL From:raw =~
> /([\w+.-]+\@[\w.-]+\.\w\w+)["'`\s]*<\s*\1>/
> +header __NAME_EQ_EMAIL From:raw =~
> /([\w+.-]+\@[\w.-]+\.\w\w+)["'`\s]*<\s*\1>/i
>  meta    NAME_EMAIL_DIFF        __NAME_IS_EMAIL && ! __NAME_EQ_EMAIL

Committing 

svn commit -m 'Bug 7167 case insentive change for NAME_EMAIL_DIFF'
Sending        khopesh/20_khop_experimental.cf
Transmitting file data .
Committed revision 1672193.

> (and rules/72_active.cf)

Isn't that what ruleqa promotion would handle?

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