Skip to content

Add Clerk secondary email records#1458

Open
jescalan wants to merge 1 commit into
Domain-Connect:masterfrom
jescalan:je/clerk-secondary-email-records
Open

Add Clerk secondary email records#1458
jescalan wants to merge 1 commit into
Domain-Connect:masterfrom
jescalan:je/clerk-secondary-email-records

Conversation

@jescalan

Copy link
Copy Markdown

Description

Add the four Clerk secondary email provider CNAME records in a new secondary_mail group. This lets Clerk include the Mailgun fallback-provider records only after secondary email provisioning completes, while preserving the existing auth and primary mail groups.

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 not be backward compatible)

How Has This Been Tested?

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

Checklist of common problems

  • syncPubKeyDomain is set
  • warnPhishing is not set alongside syncPubKeyDomain
  • syncRedirectDomain is set whenever the template uses redirect_uri in the synchronous flow
  • no TXT record contains SPF content
  • txtConflictMatchingMode is set on every TXT record that must be unique per label or content prefix
  • no variable is used as a bare full record value
  • no bare variable is used as the full host label
  • 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 or remove without breaking the template

Online Editor test results

Test clerk.com/app example.com/@

Test clerk.com/app example.com/app

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