Update byteseats.com.storefront: add Cloudflare ownership TXT record#1174
Open
nader-bytesai wants to merge 2 commits into
Open
Update byteseats.com.storefront: add Cloudflare ownership TXT record#1174nader-bytesai wants to merge 2 commits into
nader-bytesai wants to merge 2 commits into
Conversation
Adds the _cf-custom-hostname ownership-verification TXT (value via the %ownership% variable) alongside the existing CNAME, so a single one-click apply writes both records Cloudflare for SaaS requires (hostname ownership pre-validation + routing). Bumps version 2 -> 3.
|
✅ JSON Filename Check Passed |
|
✅ JSON Schema Validation Passed |
Linter OK
|
| Level | Code | Note |
|---|---|---|
| info | DCTL1021 | missing from iana definitions |
A re-apply or hostname rotation should replace the _cf-custom-hostname TXT rather than stack a duplicate, so the record stays unique.
PR Description Check PassedAll required sections are filled in correctly. Details |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Updates the previously-merged
byteseats.com.storefronttemplate (#1168) to add the Cloudflare hostname-ownership verification TXT record alongside the existing CNAME.Bytes serves customer storefronts through Cloudflare for SaaS custom hostnames, which require two DNS records to go live: the routing CNAME (already in the template) and the
_cf-custom-hostnameownership TXT (a per-hostname token that keeps ownership pre-validation on — the mitigation against dangling-CNAME subdomain takeover). With only the CNAME in the template, the one-click apply left customers to add the ownership TXT by hand; this change lets a single apply write both records.Type of change
How Has This Been Tested?
<providerId>.<serviceId>.jsonlogoUrlis actually served by a webserverChecklist of common problems
syncPubKeyDomainis set — public key is published at_dcpubkeyv1.byteseats.comwarnPhishingis not set alongsidesyncPubKeyDomainsyncRedirectDomainis set_cf-custom-hostname.<host>txtConflictMatchingModeis set on the TXT record that must be unique — set toAllso a re-apply / hostname rotation replaces the ownership token rather than stacking a duplicatepointsToand the TXT'sdataare full values (%target%/%ownership%); the request is RSA-signed (Cloudflare requires signing), preventing tamperinghostlabel — CNAME host is@; TXT host is the literal prefix_cf-custom-hostnamecombined with the applyhostparameterhostfield to create a subdomain — thehostparameter is used (hostRequired: true)%host%does not appear explicitly in anyhostattributeessential— N/A; both records are core service records that must not be removed without breaking the storefrontOnline Editor test results
Subdomain test (apex test not required since
hostRequired: true). Applying withdomain=bytesai.dev,host=order,target=qa.web.byteseats.com,ownership=<cf-token>produces:CNAME order → qa.web.byteseats.comTXT _cf-custom-hostname.order → <cf-token>Editor test link:
Test byteseats.com/storefront on bytesai.dev/order
🤖 Generated with Claude Code