fix(dns): handle split DKIM TXT records + correct smoke test curl #4045
Cross-repo impact
See what breaks downstream if this PR merges.
⮌ Merged
This pull request was merged into main.
c comment · e edit title · m merge · a approve · r request changes · ? shortcuts
Originally written by @ccantynz-alt on GitHub.
Imported from https://github.com/ccantynz-alt/AlecRae.com/pull/86
Summary
verifyDKIMonly used the first 255-char chunk of a multi-chunk TXT record returned by Porkbun, so 2048-bit RSA keys were always truncated and never matched the stored key. Fix: join all TXT entries before searching forv=DKIM1so the full key is assembled regardless of how the provider splits it.vapron-migration-type2.mdStep 8 smoke test used"from": "string"and"to": "string"— the API'sSendMessageSchemarequires"from": {email, name}and"to": [{email}]. Also corrected the endpoint path (/v1/messages/send) and expected response shape.Test plan
bun x vitest run services/dns)POST /v1/domains/:id/verifyonmail.vapron.ai— DKIM should flip toverified: true{"status":"queued"}and email delivery🤖 Generated with Claude Code
https://claude.ai/code/session_01GYakKc6rtYKmTgknsK1AiY
Generated by Claude Code