Skip to content

Update godaddy.com.aab: add SPF, DKIM, and DMARC records#1325

Merged
wdbaker-godaddy merged 4 commits into
Domain-Connect:masterfrom
rkorin-godaddy:update-godaddy-aab-email-records
Jul 6, 2026
Merged

Update godaddy.com.aab: add SPF, DKIM, and DMARC records#1325
wdbaker-godaddy merged 4 commits into
Domain-Connect:masterfrom
rkorin-godaddy:update-godaddy-aab-email-records

Conversation

@rkorin-godaddy

@rkorin-godaddy rkorin-godaddy commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description

Add email authentication records (SPF via SPFM, DKIM, DMARC) to the GoDaddy Airo App Builder template. The `%dkimSuffix%` variable captures the domain-specific DKIM identifier (e.g. `example_com.c98`) required by GoDaddy's onsecureserver.net DKIM infrastructure.

  • Add SPFM `@` — `include:secureserver.net`
  • Add CNAME `secureserver1._domainkey` → `s1.dkim.%dkimSuffix%.onsecureserver.net`
  • Add CNAME `secureserver2._domainkey` → `s2.dkim.%dkimSuffix%.onsecureserver.net`
  • Add TXT `_dmarc` — DMARC reject policy with `dmarc_rua@onsecureserver.net`
  • Updated `variableDescription` to document the `%dkimSuffix%` variable
  • Bumped version to 2

Type of change

  • New template
  • Bug fix (non-breaking change which fixes an issue in the template)
  • New feature (non-breaking change which adds functionality to the template)
  • Breaking change (fix or feature that would cause existing template behavior to be not backward compatible)

How Has This Been Tested?

  • Template functionality checked using Online Editor
  • Template file name follows the pattern `..json`
  • resource URL provided with `logoUrl` is actually served by a webserver

Checklist of common problems

  • `syncPubKeyDomain` is set — set to `domain-attach.api.godaddy.com`
  • `warnPhishing` is not set alongside `syncPubKeyDomain`
  • `syncRedirectDomain` is set whenever the template uses `redirect_uri` — N/A, no redirect_uri used
  • no TXT record contains SPF content (`"v=spf1 ..."`) — using `SPFM` record type
  • `txtConflictMatchingMode` is set on every TXT record that must be unique per label (e.g. DMARC) — not set on the `_dmarc` record; `_dmarc` is a fixed label so conflicts are unlikely, but this can be added if required
  • no variable is used as a bare full record value
  • no bare variable is used as the full `host` label — `%dkimSuffix%` only appears in `pointsTo`, not in `host`
  • no variable is used in the `host` field to create a subdomain
  • `%host%` does not appear explicitly in any `host` attribute
  • `essential` is set to `OnApply` on records the end user may need to modify — DMARC record does not set `essential`; this is intentional as GoDaddy manages the DMARC policy

Online Editor test results

Editor test link(s):
apex test
subdomain test

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added Checklist of common problems not complete See PR template and mark *all* checkboxes, even if not applicable. Explain any discrepancies. PR description incomplete The PR description template was not filled in at all, altered or filled in improperly. Test links missing No test links from Online Editor provided labels Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

JSON Filename Check Passed

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

JSON Schema Validation Passed

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Linter OK

godaddy.com.aab.json

Level Code Note

rkorin-godaddy and others added 2 commits July 6, 2026 10:38
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot removed Checklist of common problems not complete See PR template and mark *all* checkboxes, even if not applicable. Explain any discrepancies. PR description incomplete The PR description template was not filled in at all, altered or filled in improperly. Test links missing No test links from Online Editor provided labels Jul 6, 2026
- Remove ttl from SPFM record (pseudo-record, never written to DNS directly)
- Add essential/txtConflictMatchingMode/txtConflictMatchingPrefix to DMARC
  record to handle existing _dmarc records gracefully and preserve user edits

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added the Test links missing No test links from Online Editor provided label Jul 6, 2026
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

PR Description Check Failed

The PR description is missing required elements. Please update it according to the PR template.

Details
  OK  Type of change: 1/4 checkboxes ticked
  OK  How Has This Been Tested?: all 3 checkboxes ticked
  OK  Checklist of common problems: 8/10 checkboxes ticked
  OK  Online Editor test results: 2 link(s) found

Labels to add: Test links missing
Labels to remove: Checklist of common problems not complete, Forged editor links, PR description incomplete

PR description check FAILED:
  FAIL  Template coverage: godaddy.com.aab.json: 2/2 test link(s) contain a template that does not match the file in this PR (tests may be out of date)

@wdbaker-godaddy wdbaker-godaddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

approved

@wdbaker-godaddy wdbaker-godaddy merged commit 6ab1ed0 into Domain-Connect:master Jul 6, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Test links missing No test links from Online Editor provided

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants