Bug 4877 - [review] Mail::DKIM default causes DKIM_POLICY_SIGNSOME on (almost) all mail
Summary: [review] Mail::DKIM default causes DKIM_POLICY_SIGNSOME on (almost) all mail
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Plugins (show other bugs)
Version: SVN Trunk (Latest Devel Version)
Hardware: Other other
: P5 normal
Target Milestone: 3.1.2
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard: needs 2 votes
Keywords:
Depends on:
Blocks:
 
Reported: 2006-04-25 03:04 UTC by Daryl C. W. O'Shea
Modified: 2013-02-21 02:56 UTC (History)
2 users (show)


Attachment Type Modified Status Actions Submitter/CLA Status
patch - zero score for rule patch None Daryl C. W. O'Shea [HasCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daryl C. W. O'Shea 2006-04-25 03:04:59 UTC 
Mail::DKIM defaults the policy to sign some if it doesn't find a policy for a
domain.  This results in all mail, that doesn't have a policy otherwise, hitting
DKIM_POLICY_SIGNSOME.

I haven't had time (I'm in the middle of moving) to look into seeing if there's
a way to nullify this default.  If there isn't, I'd suggest we just disable the
rule (or 'meta OR' it with the other DKIM rules if people want this info).
Comment 1 Daryl C. W. O'Shea 2006-05-02 03:55:52 UTC 
Created attachment 3499 [details]
patch - zero score for rule

zero out the score for now... hopefully I'll remember to contact Jason about
the bad default in Mail-DKIM.

Sending        trunk/core/50_scores.cf
Transmitting file data .
Committed revision 398794.
Comment 2 Theo Van Dinter 2006-05-02 03:58:15 UTC 
IMO, the score zeroing can be considered "trivial" and could just be applied.  it's a new rule for 3.1, and 
it's going from being an info rule to being disabled.

+1 either way.
Comment 3 Daryl C. W. O'Shea 2006-05-02 04:00:22 UTC 
sounds good...

Sending        rules/50_scores.cf
Transmitting file data .
Committed revision 398795.
[dos@cyan 3.1]$