[Support Feedback] Improve certificate pinning, cipher suite, and Universal SSL docs (SSL/TLS)#28995
Conversation
|
/bonk review this and suggest any fixes |
![ask-bonk[bot]](https://avatars.githubusercontent.com/in/2454580?s=60&v=4){kind=small}
There was a problem hiding this comment.
One issue to address: the opening paragraph and the new Prerequisites note both mention Advanced Certificate Manager, creating redundancy. Two minor style suggestions also included. Otherwise the changes are accurate and well-structured improvements for these support-heavy topics.
...ocs/ssl/edge-certificates/additional-options/cipher-suites/customize-cipher-suites/index.mdx
Show resolved
Hide resolved
...ocs/ssl/edge-certificates/additional-options/cipher-suites/customize-cipher-suites/index.mdx
Outdated
Show resolved
Hide resolved
|
Review posted. Here's a summary of the feedback: MEDIUM (1 item):
LOW (2 items):
No build breakers, no broken links, no MDX syntax issues. All internal links resolve to existing pages. The content improvements across all three files are accurate and well-targeted at the support topics described. |
…SSL docs from support macro audit SPM-3037
…, split API sentence, fix phrasing
dmmulroy
force-pushed
the
support-macros/ssl
branch
from
7d60ada to
46ff9d6
Compare
…tation to full setup
There was a problem hiding this comment.
I'm just not really convinced about these changes to /customize-cipher-suites/index.mdx: the reason is that having CF4SaaS is not really a prerequisite for this guide in particular, and we actually point SaaS providers wanting to customize ciphers for their custom hostnames away from this page. So for this, let's maybe:
1- keep the previous version, with the shorter intro and separate note for SaaS providers
2- add notes about the ACM prereq to the specific how-to pages nested under this, instead
What do you think?
There was a problem hiding this comment.
Applied the changes. Checked in with Nic via chat.
…udflare One) (#28997) * docs(cloudflare-one): improve tunnel, WARP, and SSO troubleshooting from support macro audit SPM-3037 * docs(tunnel): move support info section to bottom per reviewer feedback (nikitacano) * fix(cloudflare-one): correct tunnel firewall requirements, update IdP navigation path * docs(tunnel): move prereqs from top intro per reviewer feedback * Revert changes to customize-cipher-suites as per #28995 * fix(cloudflare-one): address review feedback from nikitacano and ranbel - Remove duplicated 'Information to provide for support' section - Merge tunnel-specific support items (tunnel ID, cloudflared version, setup method) into the existing warp/support-ticket-best-practices partial - Keep 'Collect debug logs' section under existing 'How do I contact support?' heading - Reference the partial checklist instead of duplicating it --------- Co-authored-by: Rebecca Tamachiro <rtamachiro@cloudflare.com>
3 participants
Summary
Improves SSL/TLS documentation for common support topics, based on an audit of active support macros.
Changes
Context
These changes are driven by recurring support cases. If you want to see the underlying support data (macro frequency, case volume by topic), reach out to @dmmulroy internally.