Skip to content

Fix multiple issue relating to DMARC evaluation#513

Open
kayozaki wants to merge 3 commits intoKumoCorp:mainfrom
kayozaki:dmarcv2
Open

Fix multiple issue relating to DMARC evaluation#513
kayozaki wants to merge 3 commits intoKumoCorp:mainfrom
kayozaki:dmarcv2

Conversation

@kayozaki
Copy link
Copy Markdown
Contributor

@kayozaki kayozaki commented May 1, 2026

This is a redo of #505
The amount of conflict looked safer to start the PR over.

  • Handle parsing failure when DMARC record ends with semicolon
  • Adjust kumo.dmarc.check_msg so it considers SPF/DKIM verdict and not just alignment
  • Made kumo.dmarc.check_msg to pass, if either SPF or DKIM succeeds alignment check
  • Fix relax mode SPF/DKIM so the comparing hostnames considers top-level domain alignment
  • Made alignment comparison case insensitive
  • Added support for SPF to fallback to ehlo domain, when mail from domain is blank
  • Enrich props against dkim auth_resutls for better troubleshooting capabilities
  • Enrich props against dmarc auth_resutls for better troubleshooting capabilities

@kayozaki kayozaki changed the title Dmarcv2 Fix multiple issue relating to DMARC evaluation May 1, 2026
@kayozaki kayozaki marked this pull request as ready for review May 2, 2026 00:40
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.

1 participant