Skip to content

Switch plop.so apex template from A records to CNAME#928

Closed
ccarse wants to merge 1 commit intoDomain-Connect:masterfrom
ccarse:fix-apex-cname
Closed

Switch plop.so apex template from A records to CNAME#928
ccarse wants to merge 1 commit intoDomain-Connect:masterfrom
ccarse:fix-apex-cname

Conversation

@ccarse
Copy link
Copy Markdown
Contributor

@ccarse ccarse commented Mar 30, 2026

Description

Updates the plop.so.website-apex template to use a CNAME record at apex instead of hardcoded A record IPs. The previous A record IPs (104.21.9.9, 172.67.140.209) are Cloudflare shared proxy IPs that caused Error 1000 for domains also on Cloudflare.

Uses CNAME with `host: "@"` and `hostRequired: true` — Cloudflare applies CNAME flattening at the zone apex automatically and ignores the hostRequired flag. This follows the same pattern as PR #905 (spinlab.studio), which was confirmed by the Cloudflare Domain Connect team.

Changes:

  • Record type: 2x A records -> CNAME pointing to plop.so
  • Set hostRequired: true (required by spec for CNAME with host "@")
  • Version bumped 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 filename follows `..json` pattern
  • Resource URL with `logoUrl` is served by webserver

Checklist of common problems

  • `syncPubKeyDomain` is set in every template
  • `warnPhishing` is not used alongside `syncPubKeyDomain`
  • `syncRedirectDomain` not needed (no redirect_uri)
  • No SPF content in TXT records
  • `txtConflictMatchingMode` not needed (no TXT records)
  • No bare variables as full record values
  • No bare variables as full host labels
  • No variables creating subdomains in host field
  • `%host%` not used in host attributes
  • `essential` not needed (non-modifiable records)

Online Editor test results

plop.so.website-apex subdomain test (hostRequired=true):
https://domainconnect.paulonet.eu/dc/free/templateedit?token=H4sIAOIKy2kC%2F91SYW%2FTMBD9K5E%2FgWhKljRtEwmJbgOBEB2MMYSqKro4l86Q2MZ22oaq%2Fx27bWg7ED%2BAT8ndvXt%2B7%2B42xGAtKzBI0g2RSixZgeptQVIiKyH7WpDe7%2FQUajwUbFajWjKKO%2BwKc80M%2BiBxfSyd4L3XrELvjdCG8YX3ZGJxTy1wiUozwUka2n9QDPIKr1FTxaTZ5cmV4Byp8VrRKM%2FRe4WogXHPCO%2Bg0CuF8krH%2F7DndwpaTi8rQb%2BTtIRKo808gMLiGFrAhyZ%2Fh%2B31ju%2FMsOO5xR8N23UY1dgGhVSoQpN0tiGmlc7Y1XTy%2FtUBbsOXblSCcaPvxBmdMRVJo2EQbOfbHvkpOGZHtnmPFJ0CXINdBvapqI%2B0zpm24UKJRmasa5KgoNZua1377Kx%2F3hHMDgzubbbgQmGm7RdMo7AzVzeVYRms4JgqaAZSVq2Vqm3V8vxpnO8X3CkswMDfjfdIVlCnlblrCcskGcYx9WEMQ38QFrmfQ5D4%2BYiOyjK4CBPAk7N7dI3%2FuLtHE0OtkRsGVgSZVCtoNdlu5z07vf%2FEit2xhiUWGThsGIRDP4j8KLgLo3QwSKNBP45GyTh5FgRpEDgXqM3e2sZew%2BkdELH%2B9nEd3yw0XEwmXy5XZnz%2F9SaGaby%2BlWX4CT%2BH92paPU%2BSweIF2f4CJMAxTjgEAAA%3D

Note: Apex test (no host) is not applicable since hostRequired=true. Cloudflare ignores this flag and applies CNAME flattening at apex regardless — confirmed by Cloudflare Domain Connect team in PR #905.

- Changed record type from A (hardcoded IPs) to CNAME pointing to plop.so
- Added hostRequired: true (required by spec for CNAME with host: "@")
- Bumped version to 2
- Cloudflare supports CNAME at apex via CNAME flattening with proxied records
- Fixes Error 1000 for Cloudflare-to-Cloudflare domains

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the hostRequired hostRequired is set label Mar 30, 2026
@github-actions
Copy link
Copy Markdown

JSON Filename Check Passed

@github-actions
Copy link
Copy Markdown

JSON Schema Validation Passed

@github-actions
Copy link
Copy Markdown

Linter OK

plop.so.website-apex.json

Level Code Note

@ccarse ccarse closed this Mar 30, 2026
@github-actions github-actions bot added 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 and removed 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 Mar 30, 2026
@github-actions
Copy link
Copy Markdown

PR Description Check Passed

All required sections are filled in correctly.

Details
  OK  Type of change: 1/4 checkboxes ticked
  OK  How Has This Been Tested?: all 3 checkboxes ticked
  OK  Checklist of common problems: 10/10 checkboxes ticked
  OK  Online Editor test results: 1 link(s) found
  OK  Template coverage: all 1 template(s) covered
Labels to remove: Checklist of common problems not complete, Forged editor links, PR description incomplete, Test links missing

PR description check PASSED

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hostRequired hostRequired is set

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant