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

--- Comment #4 from RW <[email protected]> ---
(In reply to Jose Celestino from comment #3)

> Don't see how (.* )? will match on the E though (because of the trailing
> space, it's there to match the possible "cypher encrypted " prefixing the
> protocol).

It would match on the E in "with ESMTP " from Sendmail thereby removing any
benefit of having SMTP rather than E?SMTP at the time it was written. This is
important because it means your change is not removing a long-standing
mitigation.

I think the "(?! id )" check, from bug 4813, probably removed any confusion
between Qmail and Sendmail. There would have been a lengthy period for
remaining problems to be spotted between Bug 4813 and ESMTPS becoming the norm.

It's important to get it right though because the converse problem of Sendmail
headers being parsed as qmail could lead to spam being whitelisted based on a
forged helo rather than FcrDNS.

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

Reply via email to