[Bug 6420] New: [PATCH] Extend fetchmail parsing restart to cover other mail fetchers

[bu...@issues.apache.org]

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

           Summary: [PATCH] Extend fetchmail parsing restart to cover
                    other mail fetchers
           Product: Spamassassin
           Version: 3.3.1
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Rules
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: peter.alfredsen@gmail.com


Created an attachment (id=4751)
 --> (https://issues.apache.org/SpamAssassin/attachment.cgi?id=4751)
Fixup mail fetcher detection logic to be more generic

Currently, we only restart parsing of headers when encountering a Fetchmail
marker. This patch extends that to any mail received via POP3 or IMAP, which
can be assumed to have been fetched when we're in trusted_networks.

Also, initialize $self->{allow_fetchmail_markers} to 1 so we don't tell the
user in dbg that the first header is untrusted, if it is generated by a mail
fetcher.

Furthermore, remove already obsolete Received: parsing try which would have
been caught by the test this patch replaces, and is certainly caught by this
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 6420] [PATCH] Extend fetchmail parsing restart to cover other mail fetchers

[bu...@bugzilla.spamassassin.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rwmaillists@googlemail.com

--- Comment #2 from Karsten Bräckelmann <gu...@rudersport.de> ---
*** Bug 6935 has been marked as a duplicate of this bug. ***

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

[Bug 6420] [PATCH] Extend fetchmail parsing restart to cover other mail fetchers

[bu...@bugzilla.spamassassin.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kmcgrail@pccc.com

--- Comment #1 from Kevin A. McGrail <km...@pccc.com> 2011-10-29 02:57:19 UTC ---
Can anyone who uses getmail or fetchmail with SA confirm this patch works?  It
looks like it has promise but I can't confirm it much.

-- 
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 6420] [PATCH] Extend fetchmail parsing restart to cover other mail fetchers

[bu...@bugzilla.spamassassin.org]

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

--- Comment #3 from RW <rw...@googlemail.com> ---
The patch works for me.

Funny thing is that I had to dig-out an old POP3 retrieval to test the
behaviour with and without the patch. It turns out that without this patch
there is no support for "with IMAP" headers. For all these years my
internal/trusted networks would have worked correctly without any configuration
because the getmail header wasn't parsable.

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