Skip to content

More verbose nxdomain message#256

Merged
seanthegeek merged 1 commit into
domainaware:mainfrom
CERT-Polska:more-verbose-nxdomain-message
Jun 8, 2026
Merged

More verbose nxdomain message#256
seanthegeek merged 1 commit into
domainaware:mainfrom
CERT-Polska:more-verbose-nxdomain-message

Conversation

@kazet

@kazet kazet commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Let's assume we are processing an SPF record: v=spf1 mx -all for domain example.com having a single MX record of mail.example.com which doesn't exist.

Previously the error message was example.com: the domain does not exist. which is false. After this change, the message would be Error when processing example.com: the domain mail.example.com does not exist..

@kazet kazet force-pushed the more-verbose-nxdomain-message branch 3 times, most recently from 5f1ef86 to 2bf6978 Compare June 8, 2026 20:36
@kazet kazet force-pushed the more-verbose-nxdomain-message branch from 2bf6978 to fed828d Compare June 8, 2026 20:39
@codecov

codecov Bot commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.43%. Comparing base (222c098) to head (fed828d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #256   +/-   ##
=======================================
  Coverage   95.43%   95.43%           
=======================================
  Files          12       12           
  Lines        2911     2911           
=======================================
  Hits         2778     2778           
  Misses        133      133           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@seanthegeek

Copy link
Copy Markdown
Contributor

Thanks!

@seanthegeek seanthegeek merged commit b641994 into domainaware:main Jun 8, 2026
3 checks passed
seanthegeek added a commit that referenced this pull request Jun 8, 2026
### Added

- Warnings for deprecated Sender ID TXT records ([PR #255](#255))

### Changed

- `NXDOMAIN` errors are more detailed ([PR #256](#256))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants