Skip to content

[SSL] document backup certificate race condition known issue - #33218

Merged
ngayerie merged 1 commit into
productionfrom
dee-3829-backup-cert-known-issue
Sep 8, 2026
Merged

[SSL] document backup certificate race condition known issue#33218
ngayerie merged 1 commit into
productionfrom
dee-3829-backup-cert-known-issue

Conversation

@ngayerie

@ngayerie ngayerie commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Add a troubleshooting section to the backup certificates page documenting a known race condition where a backup certificate can be deleted immediately after issuance.

DEE-3829

When Universal SSL is disabled and re-enabled in quick succession, certificate processing jobs are not guaranteed to run in submission order. This can cause a newly issued backup certificate to be deleted before it becomes active. The fix documents the issue and the workaround (wait between toggling, use Advanced Certificate Manager for uninterrupted coverage).

@ngayerie
ngayerie requested review from a team, baubuchon-cf and lgarofalo as code owners September 4, 2026 11:27
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Review

✅ No issues found in commit 526bce3.

Code Review

This code review is in beta and may not always be helpful — use your judgment.

No code review issues found.

Conventions

No convention issues found.

Style Guide Review

No style-guide issues found.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.
/disable-auto-review Stops automatic reviews from triggering on future pushes to this PR. Codeowners can still run /review or /full-review manually.
/rebase Rebases the PR branch against production. On conflict, attempts to resolve automatically using AI. Stops with an explanation if confidence is not high enough.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 4, 2026

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://dee-3829-backup-cert-known-issue.previews.developers.cloudflare.com (commit 526bce3)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://a1e3c57c.previews.developers.cloudflare.com 526bce3 2026-09-08T19:23:13.181Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://6dc15f9a.previews.developers.cloudflare.com da3b83c 2026-09-05T10:00:24.044Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://d57dde20.previews.developers.cloudflare.com 209901d 2026-09-04T11:49:41.394Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://c76637b6.previews.developers.cloudflare.com f06e76e 2026-09-04T11:36:57.360Z Visit the dashboard ↗
  • Build: In progress 🔵

View logs ↗
632c831 2026-09-04T11:27:16.801Z View logs ↗

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/ssl/ @baubuchon-cf, @lgarofalo, @cloudflare/appsec-reviewers, @cloudflare/product-owners

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

@baubuchon-cf baubuchon-cf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Verified the race condition and recovery steps against the underlying escalation — matches the confirmed root cause. ACM workaround checks out too.


To recover your backup certificate:

1. Turn Universal SSL off again.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can we move the TLS continuity requirement before the recovery steps? Turning off Universal SSL can cause new TLS connections to fail unless another valid certificate is already active. “Ordering” an Advanced certificate does not guarantee it is active, and Custom certificates are also supported.

Suggested text:

Before turning off Universal SSL, make sure another valid Advanced or Custom certificate is active and covers the required hostnames. Otherwise, new TLS connections will fail.

Please also confirm that “several minutes” is the validated cooldown from DEE-3829.

@ngayerie ngayerie closed this Sep 8, 2026
@ngayerie
ngayerie force-pushed the dee-3829-backup-cert-known-issue branch from da3b83c to f3c4d7d Compare September 8, 2026 19:13
@ngayerie ngayerie reopened this Sep 8, 2026
@ngayerie
ngayerie merged commit 6a71a6e into production Sep 8, 2026
14 checks passed
@ngayerie
ngayerie deleted the dee-3829-backup-cert-known-issue branch September 8, 2026 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:ssl Related to SSL size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants