zaccone / spf

Sender Policy Framework
MIT License
19 stars 6 forks source link

Invalid evaluation of include term #25

Closed zaccone closed 7 years ago

zaccone commented 7 years ago

Parser.parseInclude() logic is slightly incorrect:

Upon recursive errors we should return Permerror, not skip the term. Recursive None should result in Permerror as well.

In general we should stick to section 5.2 in RFC7208 and rely on returnet spf.Result, not errors.