Skip to content

Probe known DKIM selectors during DNS import#15

Merged
crhaglun merged 1 commit intomainfrom
probe-dkim-selectors
May 8, 2026
Merged

Probe known DKIM selectors during DNS import#15
crhaglun merged 1 commit intomainfrom
probe-dkim-selectors

Conversation

@crhaglun
Copy link
Copy Markdown
Member

@crhaglun crhaglun commented May 7, 2026

Summary

Extract KNOWN_DKIM_PROVIDERS to shared dkim-providers.ts so both the email checks and the DNS importer use the same selector list. The importer now probes ._domainkey for TXT/CNAME records, fixing DKIM detection for providers like Proton Mail.

Test plan

Testing a domain with Proton DKIM keys now correctly displays the state as green.

  • npm test passes
  • npm run build succeeds

Checklist

  • Tests added or updated for new logic
  • No new runtime dependencies (or, if added, justified above)
  • No analytics / trackers / external embeds introduced
  • CSP connect-src updated explicitly if a new origin is needed
  • Touched files follow existing style (no as any, no @ts-ignore)

Extract KNOWN_DKIM_PROVIDERS to shared dkim-providers.ts so both
the email checks and the DNS importer use the same selector list.
The importer now probes <selector>._domainkey for TXT/CNAME records,
fixing DKIM detection for providers like Proton Mail.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@crhaglun crhaglun merged commit 3bf110d into main May 8, 2026
2 checks passed
crhaglun pushed a commit that referenced this pull request May 8, 2026
Probe known DKIM selectors during DNS import
crhaglun added a commit that referenced this pull request May 8, 2026
Probe known DKIM selectors during DNS import
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