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

             Bug #: 6759
           Summary: "ESMTPA;" not detected by Received.pm (again)
           Product: Spamassassin
           Version: 3.3.2
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Libraries
        AssignedTo: [email protected]
        ReportedBy: [email protected]
    Classification: Unclassified


r447014 reverted r157052 from bug 4184

Postfix sometimes(!?) adds a semicolon after "with ESMTPA".
r447014 should delete any ";" on the end of line but somehow it doesn't.

I patched line 391 of Received.pm

< if (/ by / && / with (ESMTPA|ESMTPSA|LMTPA|LMTPSA|ASMTP|HTTPU?)(?: |$)/i) {
> if (/ by / && / with (ESMTPA|ESMTPSA|LMTPA|LMTPSA|ASMTP|HTTPU?)(?: |$|;)/i) {

Before:
X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on
        mail.wissler-group.com
X-Spam-Flag: YES
X-Spam-Level: **********
X-Spam-Status: Yes, score=11.0 required=5.0 tests=DYN_RDNS_AND_INLINE_IMAGE,
        DYN_RDNS_SHORT_HELO_HTML,DYN_RDNS_SHORT_HELO_IMAGE,FSL_HELO_NON_FQDN_1,
       
HELO_NO_DOMAIN,HTML_MESSAGE,RCVD_IN_PBL,RCVD_IN_RP_RNBL,RCVD_IN_SEMBLACK,
        RDNS_DYNAMIC,SHORT_HELO_AND_INLINE_IMAGE,TVD_RCVD_SINGLE,
        T_DOS_OUTLOOK_TO_MX_IMAGE,T_FRT_CONTACT autolearn=no version=3.3.2
Received: from AGNBJL (ip-2-204-127-124.web.vodafone.de [2.204.127.124])
        by mail.wissler-group.com (Postfix) with ESMTPSA;
        Wed, 15 Feb 2012 09:24:57 +0100 (CET)


After:
X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on
        mail.wissler-group.com
X-Spam-Level:
X-Spam-Status: No, score=0.2 required=5.0 tests=ALL_TRUSTED,HTML_MESSAGE,
        TVD_RCVD_SINGLE,T_FRT_CONTACT autolearn=no version=3.3.2
Received: from AGNBJL (ip-2-204-127-124.web.vodafone.de [2.204.127.124])
        by mail.wissler-group.com (Postfix) with ESMTPSA;
        Wed, 15 Feb 2012 09:24:57 +0100 (CET)


Maybe this should be solved in Postfix (2.8.3). One could also use
"smtpd_sasl_authenticated_header = yes", yet what does it harm adding a test
for the semicolon in spamassassin?

Alex

-- 
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