Skip to content

Add brokerfold.com.website template#1387

Draft
searchpod wants to merge 1 commit into
Domain-Connect:masterfrom
searchpod:add-brokerfold-website-template
Draft

Add brokerfold.com.website template#1387
searchpod wants to merge 1 commit into
Domain-Connect:masterfrom
searchpod:add-brokerfold-website-template

Conversation

@searchpod

Copy link
Copy Markdown

Description

New template for BrokerFold (brokerfold.com), a real-estate website platform. The template connects a customer's domain to their BrokerFold site:

  • CNAME www → domains.brokerfold.com (routing; Cloudflare-for-SaaS edge)
  • TXT _cf-custom-hostname.www = %ownershiptxt% (Cloudflare custom-hostname ownership pre-validation; txtConflictMatchingMode: All so stale tokens from an earlier connection are replaced)
  • group apexverify: TXT _cf-custom-hostname = %apexownershiptxt% — apex ownership variant used only by support-assisted migrations; the self-serve flow applies groupId=connect only

No ACME/certificate TXTs on purpose: the flow this template serves validates certificates over HTTP once the CNAME lands, and ACME tokens rotate on retries so they don't belong in a one-shot template. Apex redirection is likewise left to the registrar's forwarding feature (not REDIR301) and is presented to the user as a separate manual step.

All apply requests are signed (syncPubKeyDomain: brokerfold.com; key published at _dcpubkeyv1.brokerfold.com, RS256).

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 EditorPR is marked DRAFT until the editor test links are attached below
  • Template file name follows the pattern <providerId>.<serviceId>.json
  • resource URL provided with logoUrl is actually served by a webserver (https://www.brokerfold.com/brokerfold-logo.svg → 200, image/svg+xml)

Checklist of common problems

  • syncPubKeyDomain is set (brokerfold.com; key live at _dcpubkeyv1.brokerfold.com)
  • warnPhishing is not set alongside syncPubKeyDomain
  • syncRedirectDomain is set (brokerfold.com) — the synchronous flow uses redirect_uri
  • no TXT record contains SPF content
  • txtConflictMatchingMode is set on every TXT record (All — the _cf-custom-hostname labels are dedicated to this single token and stale values must be replaced)
  • no variable is used as a bare full record value unless necessary — justification: %ownershiptxt% / %apexownershiptxt% are Cloudflare-for-SaaS ownership_verification tokens; the DNS provider of record (Cloudflare) requires the TXT content to equal the issued token exactly, so a fixed prefix is not possible. The hosts are fixed labels (_cf-custom-hostname.www, _cf-custom-hostname), which limits misuse, and apply requests are signed.
  • no bare variable is used as the full host label (hosts are fixed: www, _cf-custom-hostname.www, _cf-custom-hostname)
  • no variable is used in the host field to create a subdomain
  • %host% does not appear explicitly in any host attribute
  • essential is not needed — every record in the connect group is required for the service to function; the apexverify group is opt-in via groupId

Online Editor test results

Editor test link(s): to be attached before marking ready for review (this PR is a draft until then).

Validated locally against template.schema (jsonschema, no errors).

@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 labels Jul 14, 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

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  Checklist of common problems: 10/10 checkboxes ticked

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

PR description check FAILED:
  FAIL  'How Has This Been Tested?': 1/3 checkboxes are not ticked
  FAIL  'Online Editor test results': no valid editor test link found
  FAIL  Template coverage: brokerfold.com.website.json: no editor test link found (expected template id 'brokerfold.com.website')

@github-actions

Copy link
Copy Markdown

Linter OK

brokerfold.com.website.json

Level Code Note
info DCTL1021 missing from iana definitions
info DCTL1021 missing from iana definitions

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

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant