search

fastmail / mail-dkim

Mail::DKIM Perl module, forked from svn://svn.code.sf.net/p/dkimproxy/code/Mail-DKIM/trunk
8 stars 6 forks source link

ARC::Signer: Preserve leading fold from AR (if any) when copying to AAR. #12

Closed dev-aaront-org closed 4 years ago

dev-aaront-org commented 4 years ago

Currently ARC::Signer with Chain => ar will take this AR header:

Authentication-Results: example.org; 
  arc=fail (bad something);
  spf=pass smtp.mailfrom="jason\@example.net";

And produce this AAR header:

ARC-Authentication-Results: i=1; example.org; arc=fail (bad something);
  spf=pass smtp.mailfrom="jason@example.net"

This change causes ARC::Signer to copy the first fold from the AR header in to the AAR header as well, so the result is:

ARC-Authentication-Results: i=1; example.org;
  arc=fail (bad something);
  spf=pass smtp.mailfrom="jason@example.net"

This change is purely cosmetic, but I do find it very convenient to have all of the auth results indented the same. It makes it easier to read at a glance. It also seems pretty standard to fold after the authserv-id, as Mail::AuthenticationResults does, and Google seems to do for both AR and AAR headers.