postalsys / mailauth

Command line utility and a Node.js library for email authentication
Other
127 stars 10 forks source link

No RR returned if default mechanish is missing #35

Closed schack closed 1 year ago

schack commented 1 year ago

Describe the bug The RR field is not returned for SPF records where the default mechanism is missing

To Reproduce

mailauth spf -f bellcom.dk -i 1.2.3.4 { "domain": "bellcom.dk", "client-ip": "1.2.3.4", "envelope-from": "postmaster@bellcom.dk", "status": { "result": "neutral", "comment": "STL-R7N4H9RQF6: 1.2.3.4 is neither permitted nor denied by domain of postmaster@bellcom.dk", "smtp": { "mailfrom": "postmaster@bellcom.dk" } }, "header": "Received-SPF: neutral (STL-R7N4H9RQF6: 1.2.3.4 is neither permitted nor denied by domain of postmaster@bellcom.dk) client-ip=1.2.3.4;", "info": "spf=neutral (STL-R7N4H9RQF6: 1.2.3.4 is neither permitted nor denied by domain of postmaster@bellcom.dk) smtp.mailfrom=postmaster@bellcom.dk" }

However: dig +short bellcom.dk txt "v=spf1 include:_spf.bellcom.dk" "google-site-verification: mrYNF5E8kEkNacSV-zrITF03H7FSh6-qTTki86F6k84"

Expected behavior I think to remember the default mechanish is ?all (neutral), so while the record is sort of meaningless it is valid, and it would be nice if the RR field in the result was set.

andris9 commented 1 year ago

Fixed with v4.3.3

Screenshot 2023-04-13 at 10 53 32
schack commented 1 year ago

Thanks again :-)