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 2010/02/27 13:09:28 UTC

[Bug 6354] New: HK_NAME_FREE FP

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

           Summary: HK_NAME_FREE FP
           Product: Spamassassin
           Version: 3.3.0
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P5
         Component: Rules
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: mouss@netoyen.net


HK_NAME_FREE hits mail from free.fr (second french ISP) users. sample:

Return-Path: <ph...@ml.ovh.net>
X-Relay-Countries: FR FR FR FR FR ** ** FR
X-Virus-Scanned: amavisd-new at netoyen.net
X-Spam-Flag: YES
X-Spam-Score: 5.042
X-Spam-Level: *****
X-Spam-Status: Yes, score=5.042 required=5 tests=[COUNTRY_FR=0.01,
    GRDNS_IP4=0.1, HK_NAME_FREE=2.732, HTML_MESSAGE=0.001, S25R_3=2.199]
    autolearn=no
Received: from 66.mail-out.ovh.net (66.mail-out.ovh.net [91.121.185.67])
    by mx.netoyen.net (Postfix) with SMTP id 05BE3E548B4
    for <mo...@ml.netoyen.net>; Sat, 27 Feb 2010 12:43:08 +0100 (CET)
Received: (qmail 31603 invoked by uid 503); 27 Feb 2010 11:41:32 -0000
Received: from b7.ovh.net (HELO mail88.ha.ovh.net) (213.186.33.57)
  by 66.mail-out.ovh.net with SMTP; 27 Feb 2010 11:41:32 -0000
Received: from b0.ovh.net (HELO queueout) (213.186.33.50)
    by b0.ovh.net with SMTP; 27 Feb 2010 11:43:06 -0000
Mailing-List: Pour toute requete administrative, contactez php-help@ml.ovh.net;
Liste geree par ezmlm
Precedence: bulk
X-No-Archive: yes
List-Post: <ma...@ml.ovh.net>
List-Help: <ma...@ml.ovh.net>
List-Unsubscribe: <ma...@ml.ovh.net>
List-Subscribe: <ma...@ml.ovh.net>
reply-to: php@ml.ovh.net
Delivered-To: mailing list php@ml.ovh.net
Received: from b0.ovh.net (HELO queue) (213.186.33.50)
    by b0.ovh.net with SMTP; 27 Feb 2010 11:43:06 -0000
Received: from smtp3-g21.free.fr (212.27.42.3)
  by mx2.ovh.net with SMTP; 27 Feb 2010 11:43:04 -0000
Received: from smtp3-g21.free.fr (localhost [127.0.0.1])
    by smtp3-g21.free.fr (Postfix) with ESMTP id 7BBE1818119
    for <ph...@ml.ovh.net>; Sat, 27 Feb 2010 12:42:58 +0100 (CET)
Received: from zimbra5-e1.priv.proxad.net (zimbra5-e1.priv.proxad.net
[172.20.243.155])
    by smtp3-g21.free.fr (Postfix) with ESMTP id A62CD818050
    for <ph...@ml.ovh.net>; Sat, 27 Feb 2010 12:42:56 +0100 (CET)
Date: Sat, 27 Feb 2010 12:42:55 +0100 (CET)
From: carlos.de.mones@free.fr
To: php@ml.ovh.net
Message-ID:
<35...@zimbra5-e1.priv.proxad.net>
In-Reply-To: <00...@com>
MIME-Version: 1.0
Content-Type: multipart/alternative; 
    boundary="----=_Part_108537_1186906016.1267270975747"
X-Originating-IP: [82.244.229.76]
X-Mailer: Zimbra 5.0 (ZimbraWebClient - FF3.0 (Mac)/5.0.15_GA_2815.UBUNTU8_64)
X-Authenticated-User: carlos.de.mones@free.fr
X-Ovh-Tracer-Id: 380554169998234514
X-Ovh-Remote: 212.27.42.3 (smtp3-g21.free.fr)
X-Ovh-Local: 213.186.33.45 (mx2.ovh.net)
X-Spam-Check: DONE|U 0.5/N
Subject: =?utf-8?Q?Re:_[php]_d=C3=A9sinscription?=

------=_Part_108537_1186906016.1267270975747
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

J'ai le m=C3=AAme probl=C3=A8me !=20

...

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|Undefined                   |3.4.0

--- Comment #7 from Henrik Krohns <he...@hege.li> 2011-05-24 09:57:04 UTC ---
Should be fixed before 3.4

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

--- Comment #8 from Mark Martinec <Ma...@ijs.si> 2011-09-23 13:31:40 UTC ---
> Should be fixed before 3.4

Fixed code and make docs consistent between Conf.pm and PerMsgStatus.pm:

trunk:
  Bug 6354: Header ":name" modifier should only ever match a name
Sending lib/Mail/SpamAssassin/Conf.pm
Sending lib/Mail/SpamAssassin/PerMsgStatus.pm
Committed revision 1174742.

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

mouss <mo...@netoyen.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mouss@netoyen.net

--- Comment #3 from mouss <mo...@netoyen.net> 2010-02-27 13:20:06 UTC ---
(In reply to comment #2)
> Yep I would say that is certainly a bug. Some other rules also seem to assume
> it would only hit the name portion.

(In reply to comment #2)
> Yep I would say that is certainly a bug. Some other rules also seem to assume
> it would only hit the name portion.

I concur. tested a
    header TST_OVH  From:name =~ /\bovh\b/mi
and it hits
From: support@ovh.com


That said, even if this is fixed, HK_NAME_FREE will hit mail where users put
there ISP in the display name:

From: joe \(free\) <jo...@free.fr>

as well as

From: joe@free.fr <jo...@free.fr>

Let me grep my mail for such things...

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

--- Comment #10 from Mark Martinec <Ma...@ijs.si> 2011-09-23 13:37:15 UTC ---
removed a debugging aid (Bug 6354)
  Sending lib/Mail/SpamAssassin/PerMsgStatus.pm
Committed revision 1174746.

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

Karsten Bräckelmann <gu...@rudersport.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P5                          |P3
           Severity|enhancement                 |normal

--- Comment #4 from Karsten Bräckelmann <gu...@rudersport.de> 2010-02-27 13:47:43 UTC ---
Not a regression, though. :/  In the absence of a real name, From:name matches
against the address. Same with 3.2.

This might even be correct, though unexpected, according to the documentation.
"Appending :name to the header name will cause everything except the first real
name to be removed from the header."

Priority P3, Severity normal.

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

--- Comment #9 from Mark Martinec <Ma...@ijs.si> 2011-09-23 13:32:19 UTC ---
Created attachment 4970
  --> https://issues.apache.org/SpamAssassin/attachment.cgi?id=4970
proposed patch

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

[Bug 6354] HK_NAME_FREE FP

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

--- Comment #1 from Karsten Bräckelmann <gu...@rudersport.de> 2010-02-27 12:29:59 UTC ---
(In reply to comment #0)
> HK_NAME_FREE hits mail from free.fr (second french ISP) users. sample:

> From: carlos.de.mones@free.fr

Uh. The real problem here seems to be that in the case of an actually missing
real name, From:name matches against the address, which it is not supposed to.

  header HK_NAME_FREE  From:name =~ /\b(?:get)?free\b/mi

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

--- Comment #6 from Karsten Bräckelmann <gu...@rudersport.de> 2010-03-01 22:15:33 UTC ---
I didn't mean to justify current behavior, and absolutely agree that it is a
bug. I wouldn't have bumped the Priority otherwise.

(That part of comment 4 was thinking out loud, whether splitting RFC hairs in
order to come up with a smart-ass definition by which it possibly could be
interpreted being correct would be possible.)

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

--- Comment #5 from Mark Martinec <Ma...@ijs.si> 2010-03-01 19:31:32 UTC ---
> Not a regression, though. :/  In the absence of a real name, From:name matches
> against the address. Same with 3.2.
> 
> This might even be correct, though unexpected, according to the documentation.
> "Appending :name to the header name will cause everything except the first real
> name to be removed from the header."

I can't see how that description could be interpreted as "in absence of a
real name, From:name matches against the address". I think the behaviour is
plain wrong and needs to be fixed to match the documentation, even if risking
bug-for-bug-compatibility.

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kmcgrail@pccc.com
           Severity|normal                      |blocker

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

Mark Martinec <Ma...@ijs.si> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #11 from Mark Martinec <Ma...@ijs.si> 2011-10-04 13:28:37 UTC ---
Closing. Fixed for trunk/3.4.0.

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

[Bug 6354] Header ":name" modifier should only ever match a name

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hege@hege.li
            Summary|HK_NAME_FREE FP             |Header ":name" modifier
                   |                            |should only ever match a
                   |                            |name

--- Comment #2 from Henrik Krohns <he...@hege.li> 2010-02-27 13:05:54 UTC ---

Yep I would say that is certainly a bug. Some other rules also seem to assume
it would only hit the name portion.

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