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

Kent Oyer <k...@mxguardian.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #5989|0                           |1
        is obsolete|                            |

--- Comment #10 from Kent Oyer <k...@mxguardian.net> ---
Created attachment 5991
  --> https://bz.apache.org/SpamAssassin/attachment.cgi?id=5991&action=edit
Revised patch

You are talking about verifying the ARC chain which is already done by the time
this code gets executed. That is handled by Mail::DKIM::ARC::Verifier in the
DKIM plugin. I'm looking at lines 330-408 in the DMARC plugin which involves
choosing which ARC headers (i.e. sealers) we want to trust and pulling the
appropriate data from them.

I think it's safe to say the current implementation is flawed but fixing it is
going to require a deeper discussion of ARC and some samples that actually
contain ARC headers. 

This bug report is about a specific error you encountered on an email that
doesn't have ARC headers. I've committed the attached patch in revision 1922739
which should prevent the error from happening again. 

If you need ARC support and have some samples of it not working, please feel
free to open a new bug report. Otherwise I think we can close this case.

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

Reply via email to