CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

fix(dns): handle split DKIM TXT records + correct smoke test curl #4045

Merged⚡ AI-generatedXSccantynz wants to mergeclaude/ecstatic-hopper-77iamvmainopened Jun 20, 20260/3 tasks
ccantynzcommented Jun 20, 2026

Originally written by @ccantynz-alt on GitHub.
Imported from https://github.com/ccantynz-alt/AlecRae.com/pull/86


Summary

  • fix(dns): verifyDKIM only 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 for v=DKIM1 so the full key is assembled regardless of how the provider splits it.
  • fix(docs): vapron-migration-type2.md Step 8 smoke test used "from": "string" and "to": "string" — the API's SendMessageSchema requires "from": {email, name} and "to": [{email}]. Also corrected the endpoint path (/v1/messages/send) and expected response shape.

Test plan

  • DNS service: 13 auto-config tests pass (bun x vitest run services/dns)
  • Re-run POST /v1/domains/:id/verify on mail.vapron.ai — DKIM should flip to verified: true
  • Run smoke test with corrected curl — expect {"status":"queued"} and email delivery

🤖 Generated with Claude Code

https://claude.ai/code/session_01GYakKc6rtYKmTgknsK1AiY


Generated by Claude Code

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts