Skip to content

[docs] Fix 3.x LDAP authentication guide links - #4583

Open
VictorMoni wants to merge 2 commits into
apache:masterfrom
VictorMoni:docs/fix-3x-ldap-authentication-links
Open

VictorMoni wants to merge 2 commits into
apache:masterfrom
VictorMoni:docs/fix-3x-ldap-authentication-links

Conversation

@VictorMoni

Copy link
Copy Markdown
Contributor

Summary

The 3.x Authentication and Authorization overview links to ./ldap.md, but the English document lives at ./authentication/ldap.md. Following the overview's LDAP link leads to the missing page reported in #4189.

Point the English link at the existing LDAP guide and align its Chinese counterpart with the same guide used by the 3.x sidebar. The Chinese tree still contains a legacy auth/ldap.md; the new link selects the canonical auth/authentication/ldap.md page.

Related to #4189.

Versions

  • dev
  • 4.x
  • 3.x
  • 2.1 or older

Languages

  • Chinese
  • English

Validation and review

  • Confirmed the former English target does not exist.
  • Both corrected links resolve to existing documents using the repository's resolveInternalTarget implementation and match the LDAP document in the 3.x sidebar.
  • Ran the repository link checker for the two changed files. It reports eight pre-existing findings involving other links/anchors; verified that those source lines and relevant heading sets are unchanged by this PR.
  • git diff HEAD^ --check passes.
  • Reviewed current/dev and 4.x counterparts: they already use ./authentication/ldap.md.
  • Scope: two link destinations only. No page moves, slug, sidebar, routing, UI, or authentication behavior changes. No additional correctness or navigation issues found in the change.
  • Full site build not run for this link-only edit.

Docs Checklist

  • Checked by AI
  • Test Cases Built
  • Updated required version and language counterparts, or explained why not
  • If only one language changed, confirmed whether source/translation counterparts need sync

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant