[SSL] Revise and consolidate client certs content #23365
Merged
Conversation
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
Contributor
|
Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment. We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally. PR Change SummaryRevised and consolidated content related to client certificates and mutual TLS (mTLS) across multiple documentation pages.
Modified Files
How can I customize these reviews?Check out the Hyperlint AI Reviewer docs for more information on how to customize the review. If you just want to ignore it on this PR, you can add the Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add |
github-actions
bot
added
product:api-shield
product:learning-paths
product:ssl
Contributor
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
Contributor
This reverts commit 721562e.
RebeccaTamachiro
requested review from
a team and
patriciasantaana
as code owners
Contributor
|
This PR requires additional review attention because it affects the following areas: PartialsThis PR updates partial files, which are pieces of content used across multiple files in our Render component.
|
mia303
reviewed
Jul 18, 2025
mia303
reviewed
Jul 18, 2025
src/content/docs/learning-paths/mtls/mtls-app-security/related-features.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/learning-paths/mtls/mtls-app-security/index.mdx
Outdated
Show resolved
Hide resolved
marciocloudflare
approved these changes
Jul 21, 2025
src/content/docs/ssl/client-certificates/create-a-client-certificate.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: marciocloudflare <[email protected]>
sdnts
pushed a commit
to sdnts/cloudflare-docs
that referenced
this pull request
Jul 24, 2025
* Add (mTLS) to page title * Review glossary entry and apply mTLS definition throughout * Link to BYOCA and learning path from client certs landing page * Review content in byo-ca and adjust page intro * Remove step-like sections and add new proposed h2s * Fill in Scope section and delete redundant partial * Add placeholder text and hyperlinks under #use-cases * Make CA distiction less wordy and use parallelism for clarity * Add horizontal break line and DirectoryListing to index.mdx * Review links out to create-a-client-certificate * Revise create-a-client-cert for wordiness and consistency with LP * Text refinement and remove Workers from #next-steps * Reorder pages * Separate forward option from enable-mtls page * Adjust heading and hyperlinks text * Explicitly call out only one CA per hostname association * Revert "Explicitly call out only one CA per hostname association" This reverts commit 721562e. * Add intro paragraph and change h2 to how-it-works * Text review * Review enable-mtls leveraging LP and new content in this PR * Replace placeholder in #use-cases and add more info about workers * General text review * Explicitly mention and link to BYOCA from LP and #use-cases * Fix broken anchors * Fix info on requests including Cf-Client-Cert headers * Text review * Add section about multiple CAs for the same hostname * Remove DirectoryListing from overview page * Remove additional warning from enable-mtls and delete partial * Apply changes following PM review * Remove remaining mention of account-specific CA * Improve consistency for hyperlinks to learning path Co-authored-by: marciocloudflare <[email protected]> * Fix capitalization --------- Co-authored-by: marciocloudflare <[email protected]>
thomasgauvin
pushed a commit
that referenced
this pull request
Aug 15, 2025
* Add (mTLS) to page title * Review glossary entry and apply mTLS definition throughout * Link to BYOCA and learning path from client certs landing page * Review content in byo-ca and adjust page intro * Remove step-like sections and add new proposed h2s * Fill in Scope section and delete redundant partial * Add placeholder text and hyperlinks under #use-cases * Make CA distiction less wordy and use parallelism for clarity * Add horizontal break line and DirectoryListing to index.mdx * Review links out to create-a-client-certificate * Revise create-a-client-cert for wordiness and consistency with LP * Text refinement and remove Workers from #next-steps * Reorder pages * Separate forward option from enable-mtls page * Adjust heading and hyperlinks text * Explicitly call out only one CA per hostname association * Revert "Explicitly call out only one CA per hostname association" This reverts commit 721562e. * Add intro paragraph and change h2 to how-it-works * Text review * Review enable-mtls leveraging LP and new content in this PR * Replace placeholder in #use-cases and add more info about workers * General text review * Explicitly mention and link to BYOCA from LP and #use-cases * Fix broken anchors * Fix info on requests including Cf-Client-Cert headers * Text review * Add section about multiple CAs for the same hostname * Remove DirectoryListing from overview page * Remove additional warning from enable-mtls and delete partial * Apply changes following PM review * Remove remaining mention of account-specific CA * Improve consistency for hyperlinks to learning path Co-authored-by: marciocloudflare <[email protected]> * Fix capitalization --------- Co-authored-by: marciocloudflare <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Summary
PCX-11030
PCX-17021
PCX-14675
Documentation checklist