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

--- Comment #13 from Mark Martinec <[email protected]> ---
I think the patch should be restricted to this particular case, not
inappropriately considering the VIA clause as a protocol for everybody.

+  # via $PROTOCOL is used by froufrou mailers like United Internet and
     shouldn't cause issues to allow - see bug 7101

This comment is wrong, they are adding a comment, not a via clause.

-  if (/ by / && / with ((?:ES|L|UTF8S|UTF8L)MTPS?A|ASMTP|HTTPU?)(?: |;|$)/i) {
+  if (/ by / && / (?:with|via) ((?:ES|L|UTF8S|UTF8L)MTPS?A|ASMTP|HTTPU?)(?:
|;|$)/i) {

It should probably be limited to their use-case:  (?:with|\(via)

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

Reply via email to