mirrors/maddy
1
0
Fork 0
mirror of https://github.com/foxcpp/maddy.git synced 2025-04-05 14:07:38 +03:00
Code Issues Projects Releases Packages Wiki Activity

check/dkim: Don't replace reason with "body limit used" on error

Browse source 
Additional checks were executed on broken signatures too.
This commit is contained in:
fox.cpp 2019-10-21 02:25:18 +03:00
parent 21d842ba3e
commit ac7f4d5bc1
No known key found for this signature in database
GPG key ID: E76D97CCEDE90B6C
1 changed files with 12 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

18
check/dkim/dkim.go
View file

@ -149,12 +149,8 @@ func (d dkimCheckState) CheckBody(header textproto.Header, body buffer.Buffer) m
res := module.CheckResult{AuthResult: make([]authres.Result, 0, len(verifications))}
for _, verif := range verifications {
var val authres.ResultValue
if verif.Err == nil {
goodSigs = true
d.log.Debugf("good signature from %s (%s)", verif.Domain, verif.Identifier)
val = authres.ResultPass
} else {
val := authres.ResultValue(authres.ResultPass)
if verif.Err != nil {
val = authres.ResultFail
d.log.Printf("%v (domain = %s, identifier = %s)", strings.TrimPrefix(verif.Err.Error(), "dkim: "), verif.Domain, verif.Identifier)
if dkim.IsPermFail(err) {
@ -163,8 +159,18 @@ func (d dkimCheckState) CheckBody(header textproto.Header, body buffer.Buffer) m
if dkim.IsTempFail(err) {
val = authres.ResultTempError
}
res.AuthResult = append(res.AuthResult, &authres.DKIMResult{
Value: val,
Domain: verif.Domain,
Identifier: verif.Identifier,
})
continue
}
goodSigs = true
d.log.Debugf("good signature from %s (%s)", verif.Domain, verif.Identifier)
signedFields := make(map[string]struct{}, len(verif.HeaderKeys))
for _, field := range verif.HeaderKeys {
signedFields[nettextproto.CanonicalMIMEHeaderKey(field)] = struct{}{}