Skip to content

Warn that cnameStrategy Follow also follows wildcard-synthesized CNAMEs - #2230

Merged
cert-manager-prow[bot] merged 2 commits into
cert-manager:masterfrom
wallrj:dns01-wildcard-cname-warning
Aug 20, 2026
Merged

Warn that cnameStrategy Follow also follows wildcard-synthesized CNAMEs#2230
cert-manager-prow[bot] merged 2 commits into
cert-manager:masterfrom
wallrj:dns01-wildcard-cname-warning

Conversation

@wallrj

@wallrj wallrj commented Aug 10, 2026

Copy link
Copy Markdown
Member

Adds a warning to the "Delegated Domains for DNS01" section: with cnameStrategy: Follow, cert-manager also follows CNAMEs synthesised by wildcard records (RFC 4592), because they are indistinguishable from explicit ones in a DNS response. A wildcard CNAME covering _acme-challenge.<domain> therefore sends the challenge TXT record to the wildcard target, which the DNS provider rejects as out of zone.

This has caught several users; see the diagnosis in cert-manager/cert-manager#5716 (comment: cert-manager/cert-manager#5716 (comment)) and the same failure with Cloudflare in cert-manager/cert-manager#5751.

Note that cert-manager/cert-manager#8639 proposes a code fix for #5751: detect a wildcard-synthesised CNAME (by querying *.<domain> directly and comparing targets) and skip following it. Until something like that merges, this warning documents the behaviour of all released versions.

with claude fable-5

A wildcard CNAME covering _acme-challenge.<domain> is followed like an
explicit one, so cert-manager tries to create the challenge TXT record at
the wildcard target and the DNS provider rejects it as out of zone.

See cert-manager/cert-manager#5716

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Richard Wall <richard@the-moon.net>
@cert-manager-prow cert-manager-prow Bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 10, 2026
@netlify

netlify Bot commented Aug 10, 2026

Copy link
Copy Markdown

Deploy Preview for cert-manager ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit b6e6dea
🔍 Latest deploy log https://app.netlify.com/projects/cert-manager/deploys/6a874bfce70c0a000874bbc4
😎 Deploy Preview https://deploy-preview-2230--cert-manager.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI 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.

Pull request overview

Adds documentation to warn users that cnameStrategy: Follow will also follow wildcard-synthesized CNAME answers (RFC 4592), which can cause DNS01 TXT updates to be attempted in an out-of-zone target and fail.

Changes:

  • Add a warning block explaining wildcard-synthesized CNAME behavior with cnameStrategy: Follow.
  • Provide an example failure mode and suggested mitigations (explicit _acme-challenge.<domain> record or cnameStrategy: None).
Suppressed comments (1)

content/docs/configuration/acme/dns01/README.md:148

  • The Route 53 error message is wrapped in an inline code span that contains a newline. Inline code spans cannot contain newlines in Markdown, so this is likely to render incorrectly. Keep the backticked text on a single line (or use a fenced code block inside the blockquote).
> challenge TXT record at `lb.example.org` — typically failing with an
> error such as Route 53's `RRSet with DNS name lb.example.org. is not
> permitted in zone example.com.`. To avoid this, create an explicit record

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +142 to +144
> found at `_acme-challenge.<domain>`, including one synthesized by a wildcard
> record. For example, a record like `*.example.com IN CNAME
> lb.example.org` also answers CNAME queries for

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed in b6e6dea — reflowed the blockquote so each code span stays on one line (both occurrences).

with claude fable-5

Copilot review flagged that two code spans in the blockquote were split
across lines, which can render incorrectly.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Richard Wall <richard@the-moon.net>
@hjoshi123

Copy link
Copy Markdown
Contributor

/lgtm
/approve

@cert-manager-prow cert-manager-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 20, 2026
@cert-manager-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hjoshi123

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cert-manager-prow cert-manager-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 20, 2026
@cert-manager-prow
cert-manager-prow Bot merged commit a389455 into cert-manager:master Aug 20, 2026
7 checks passed
@wallrj
wallrj deleted the dns01-wildcard-cname-warning branch August 21, 2026 05:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants