RS: Improve role-based LDAP guide with how-it-works, terminology, and validation steps - #3722
Open
kaitlynmichael wants to merge 1 commit into
Open
RS: Improve role-based LDAP guide with how-it-works, terminology, and validation steps#3722kaitlynmichael wants to merge 1 commit into
kaitlynmichael wants to merge 1 commit into
Conversation
Contributor
Contributor
Contributor
🧠 Redis MemoryNo related prior context found — this looks new to the repository's memory. Memory updated at 3b85038 |
dwdougherty
approved these changes
Jul 28, 2026
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
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
Rewrites the Redis Software Enable role-based LDAP page from a field reference into a self-service operator guide, so a customer can understand each setting and validate it before saving.
Tickets
Delivers three related tickets in one page:
What changed
%u,%D)ldapsearchcheck per setting, plus an end-to-end checkcd=admin→cn=admin; group filter(members=%D)→(member=%D)Behavior verified against the Redis Software 8.2.1-2 source.
Review notes
Security SME review requested from @aharonblitzer. Two points to confirm:
AppendCertsFromPEM, which parses multiple concatenated PEM certs, so a bundle appears to be accepted. No "no bundles" limitation was documented; please confirm current behavior before resolving DOC-2434.Note
Low Risk
Documentation-only changes to LDAP configuration guidance; no product code or runtime behavior is modified.
Overview
Expands the Redis Software Enable role-based LDAP page from a short UI walkthrough into a fuller operator guide.
The page now explains the bind → authenticate → authorize flow, adds an LDAP terminology table (DN,
%u,%D, scope, and related terms), and clarifies when to use Template vs Query for authentication and Attribute vs Query for authorization. LDAP server, trusted CA, and bind credentials sections are rewritten with clearer semantics (including PEM CA guidance and hostname verification).A major addition is Validate your LDAP configuration: step-by-step
ldapsearchexamples for protocol/host/TLS, bind user, auth query, authz query, and LDAP mapping DN matching, plus an end-to-end sign-in check. The front matter description now mentions validation; setup steps point readers to validate before Save & Enable.Minor doc fixes: bind DN example
cd=admin→cn=admin, group filter example(members=%D)→(member=%D).