domainaware / checkdmarc

A parser for SPF and DMARC DNS records
https://domainaware.github.io/checkdmarc
Apache License 2.0
251 stars 77 forks source link

The SPFSyntaxError exception isn't raised when text after "all" tag exists #73

Closed stereohj closed 1 year ago

stereohj commented 2 years ago

Hello! When is used the parse_spf_record (or check_domains) function with an spf record value with text after "all" tag, SPFSyntaxError Exception isn't being raised.

Pascal76 commented 2 years ago

The same when there is a redirect before an other mechanism ex: v=spf1 redirect=spf.campaign.adobe.com ip4:151.106.144.120 It should return a warning like: The modifiers ('redirect=spf.campaign.adobe.com') should be after all the mechanisms

seanthegeek commented 1 year ago

Anything treated after the all mechanism is ignored according to the RFC. It is atted as a warning, not an exception or error, because the previous half of the record is still usable.

Pascal76 commented 1 year ago

and what about my message @seanthegeek ?