DOC-7118 unit 4: convert rc security/ to render hooks - #4120
Merged
Merged
Conversation
Migrate content/operate/rc/security/ off the relref and note/warning/tip/info shortcodes, to the DOC-6909 render-hook equivalents (plain Markdown links and `> [!NOTE]` blockquotes), continuing the site-wide rollout. Ran build/migrate_shortcode_links.py on all 31 files in this unit (29 changed; 2 had no shortcode syntax to convert). Converted 149 relref instances and 52 callout shortcodes. Hand-fixed beyond the automated pass: - 18 nested callouts (inside list items, indent levels 1-2 deep) where the converter only indents the blockquote header line, detaching continuation lines from the enclosing <li> and leaving a spurious trailing `>` line at the old closing-tag position. Fixed across configure-acls.md, active-active-roles.md, create-assign-users.md, tls-ssl.md, private-service-connect.md (x3), saml-integration-auth0.md, saml-integration-azure-ad.md (x3), saml-integration-okta-generic.md (x4), saml-integration-okta-org2org.md, and saml-sso/_index.md (x2). Verified against rendered HTML, not just source diff, that each note/warning now nests correctly inside its <li>. - One malformed closing tag `{{</ note >}}` (space after slash) in saml-integration-auth0.md, normalized before running the converter so it wouldn't silently survive unconverted. - One bare-`/commands` link (configure-acls.md) caught by check_uncanonicalized_links.py --fix; 0 fixable/mount-only/dead findings remain afterward. Verified with a full `hugo` build (no --minify) of both origin/main and this branch: content/operate/rc has a pre-existing, unrelated fatal render error on content/develop/clients/hiredis/_index.md (jupyter-example shortcode, missing examples.json data) that occurs identically on both sides, so it doesn't block the comparison. diff_rendered_hrefs.py over operate/rc reports 0 href diffs across all 277 pages. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at 6e388ec |
5 tasks
7 tasks
5 tasks
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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
Unit 4 of the DOC-7118 rollout: migrates
content/operate/rc/security/(31 files) off Hugorelrefandnote/warning/tip/infocallout shortcodes to the DOC-6909 render-hook equivalents (plain Markdown links +> [!NOTE]blockquotes). Same pattern as wave 1 (units 1-2, #4118/#4119) and every other product directory already converted undercontent/operate/andcontent/develop/.build/migrate_shortcode_links.py.<li>and a spurious trailing>line — verified the fix against rendered HTML<li>nesting, not just the source diff.{{</ note >}}, space after slash) insaml-integration-auth0.mdbefore conversion so it wouldn't silently survive unconverted.build/check_uncanonicalized_links.py --fixscoped to this unit's files: found and fixed 1 bare/commandslink inconfigure-acls.md; confirmed 0 FIXABLE/MOUNT_ONLY/DEAD remain.#anchorcross-references intosecurity/access-control/pages resolve to real headings.Verification
Built
public/fromorigin/mainand from this branch with realhugo(no--minify),npm installrun in both worktrees first. Both builds hit the same pre-existing, unrelated fatal render error oncontent/develop/clients/hiredis/_index.md(ajupyter-exampleshortcode missing generatedexamples.jsondata) — identical on both sides, so it doesn't affect the comparison; bothpublic/trees were still fully populated for all pages underoperate/rc.build/diff_rendered_hrefs.pycomparing before/after over theoperate/rcprefix: 0 href diffs across all 277 pages.Ref: DOC-7118
🤖 Generated with Claude Code
Note
Low Risk
Documentation-only syntax migration with verified equivalent rendered links; no application or security product behavior changes.
Overview
DOC-7118 unit 4 updates
content/operate/rc/security/(~31 files) so internal links and admonitions use the DOC-6909 render-hook style instead of Hugo.Links:
{{< relref "..." >}}becomes explicit/content/...Markdown paths (including_index.mdwhere needed and preserved#anchorfragments).Callouts:
note/warning/tipshortcodes become GitHub-style blockquotes (> [!NOTE],> [!WARNING],> [!TIP]). Embedded shortcodes such as{{< embed-md "..." >}}and images are unchanged.Coverage spans the security hub, access control (RBAC, MFA, SAML IdP guides), database security, and network connectivity (VPC peering, PrivateLink, Transit Gateway, PSC, CIDR). The PR description notes bulk conversion via
migrate_shortcode_links.py, hand fixes for nested callouts in lists, one/commandslink fix, and 0 href diffs in renderedoperate/rcoutput—so this is a markup migration, not a content rewrite.Reviewed by Cursor Bugbot for commit 6e388ec. Bugbot is set up for automated code reviews on this repo. Configure here.