Compare commits

..

No commits in common. "9c3109c0877e4fca1c79860a358f2248037b9f34" and "08ef2151e3841e7125f9e2049e9f276e881f74ab" have entirely different histories.

2 changed files with 3 additions and 14 deletions

View file

@ -58,7 +58,4 @@ score RCVD_IN_VALIDITY_RPBL_BLOCKED 0
score RCVD_IN_VALIDITY_SAFE_BLOCKED 0
score RCVD_IN_VALIDITY_CERTIFIED 0
score RCVD_IN_VALIDITY_RPBL 0
score RCVD_IN_VALIDITY_SAFE 0
# emails with no text are common for tests, don't penalize them
score EMPTY_MESSAGE 0
score RCVD_IN_VALIDITY_SAFE 0

View file

@ -302,9 +302,7 @@ func (r *DNSListChecker) reverseIP(ipStr string) string {
}
// CalculateScore calculates the list contribution to deliverability.
// Informational lists don't count proportionally; instead, if any
// informational list triggers, a flat 10% penalty is applied regardless
// of how many of them fire.
// Informational lists are not counted in the score.
func (r *DNSListChecker) CalculateScore(results *DNSListResults, forWhitelist bool) (int, string) {
scoringListCount := len(r.Lists) - len(r.informationalSet)
@ -326,13 +324,7 @@ func (r *DNSListChecker) CalculateScore(results *DNSListResults, forWhitelist bo
return 100, "A+"
}
// A listing on any informational list applies a flat 10% penalty.
informationalPenalty := 0
if results.ListedCount > results.RelevantListedCount {
informationalPenalty = 10
}
percentage := max(0, 100-results.RelevantListedCount*100/scoringListCount-informationalPenalty)
percentage := 100 - results.RelevantListedCount*100/scoringListCount
return percentage, ScoreToGrade(percentage)
}