|
SA Bugzilla – Full Text Bug Listing |
Summary: | Mail::SpamAssassin::Plugin::SPF should have support for Authentication-Results headers | ||
---|---|---|---|
Product: | Spamassassin | Reporter: | Benny Pedersen <me> |
Component: | Plugins | Assignee: | SpamAssassin Developer Mailing List <dev> |
Status: | RESOLVED DUPLICATE | ||
Severity: | normal | CC: | apache, kmcgrail, me, vesely |
Priority: | P2 | ||
Version: | unspecified | ||
Target Milestone: | Undefined | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: |
Description
Benny Pedersen
2013-01-25 21:24:40 UTC
As a general rule, SA doesn't add extra headers. This information might be suitable for the report information, however. its more likely that i forgot to say Mail::SPF is missing it also, so it depends how spf plugin is configured overall, default is that is unstable in how its tested now, but i created this bug to get it common on all servers around the world, its like if there is more then one mouse hold the cat needs to find more then one hold :=) (In reply to comment #2) > its more likely that i forgot to say Mail::SPF is missing it also, so it > depends how spf plugin is configured overall, default is that is unstable in > how its tested now, but i created this bug to get it common on all servers > around the world, its like if there is more then one mouse hold the cat > needs to find more then one hold :=) SA is more of an API for testing things and this is more of something at the MTA level. Sorry, but I would not support SA just adding more and more headers to support more and more features (DKIM, SPF, RBLs, etc). I do think the information would be useful to add to the report. Overall though if you can get a patch together or convince someone to write a patch, I would likely not stand in it's way. its reuse of AR header like its already done with recieved-spf header i like to have supported in spf plugin Benny and I have been talking and apparently this is older functionality that needs to be ported forward to the new SPF implementation. Do we have anything in SA that uses an existing SPF header and avoids a second SPF lookup? (In reply to comment #5) > Do we have anything in SA that uses an existing SPF header > and avoids a second SPF lookup? Mail::SpamAssassin::Plugin::SPF uses Received-SPF:, and has documented options to tweak that usage. Authentication-Results: can be used to report SPF results as an alternative, or in addition, to Received-SPF:. As Authentication-Results: can be used to report various results, it is better to factor parsing into a new plugin, and then have each module check whether the result it wants is already in. |