Skip to content

failing DKIM testcase#52

Open
link2xt wants to merge 2 commits intomainfrom
link2xt/rnlpkmxnlstp
Open

failing DKIM testcase#52
link2xt wants to merge 2 commits intomainfrom
link2xt/rnlpkmxnlstp

Conversation

@link2xt
Copy link
Contributor

@link2xt link2xt commented Feb 17, 2026

I have hardcoded DKIM key in case it changes

Original for reference:
$ dig +short TXT default._domainkey.abjadiyah.xyz
"v=DKIM1; h=sha256; k=rsa; " "p=MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA5krC4Xi5Wkr6eMlla38LCFmV645E3FLAgsRl2YJ0SrZ4N2Vw1/yH0mefvtk7HYE7ytV7RQl/er2CkSsaHLJSYLmPCBw5CO6PSsBSXuh6DBqdylh/1t9vVQ9p38fTwn9gU1QvplcpRQL9eepRra1k24VMIaVy2ZZcu3LI9zkPsR7o7TyNaeMhsL8ouWInWc1NSid+p0SgliQuwH" "IejZhlTPE60JLbJE0OR9I4wmq3377H6z/QrO8XeabCgtmTuzE/hTRyIyNS40jql/99pjlhIcjM2U+P2B0FjwYt7BwLHsgANr74ctlnKY+SdH25rNwVpPmkotaULG5SJCByKBkfCwIDAQAB"

I have hardcoded DKIM key in case it changes

Original for reference:
$ dig +short TXT default._domainkey.abjadiyah.xyz
"v=DKIM1; h=sha256; k=rsa; " "p=MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA5krC4Xi5Wkr6eMlla38LCFmV645E3FLAgsRl2YJ0SrZ4N2Vw1/yH0mefvtk7HYE7ytV7RQl/er2CkSsaHLJSYLmPCBw5CO6PSsBSXuh6DBqdylh/1t9vVQ9p38fTwn9gU1QvplcpRQL9eepRra1k24VMIaVy2ZZcu3LI9zkPsR7o7TyNaeMhsL8ouWInWc1NSid+p0SgliQuwH" "IejZhlTPE60JLbJE0OR9I4wmq3377H6z/QrO8XeabCgtmTuzE/hTRyIyNS40jql/99pjlhIcjM2U+P2B0FjwYt7BwLHsgANr74ctlnKY+SdH25rNwVpPmkotaULG5SJCByKBkfCwIDAQAB"
return Err("500 Failed to parse message headers".to_string());
};

let mut viadkim_headers: Vec<HeaderField> = Vec::new();
Copy link
Contributor Author

@link2xt link2xt Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using mailparse to split headers from the body and parse them is overkill, example from viadkim simply splits on the first \r\n\r\n and then using viadkim parsing: #52

The problem is likely entirely in viadkim, in this case the test can go there and we need to fork it.

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