Skip to content

Conversation

@Meggielqk
Copy link
Collaborator

@Meggielqk Meggielqk added this to the 5.8.9 milestone Oct 22, 2025
@Meggielqk Meggielqk requested a review from Copilot October 22, 2025 06:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds documentation for the username field in Redis authentication and authorization configurations, supporting Redis ACL (Access Control List) functionality introduced in Redis 6.0.

Key Changes:

  • Added comprehensive documentation for the new username field in Redis authentication and authorization
  • Reorganized configuration sections to distinguish between Dashboard configuration and configuration file methods
  • Updated instructions for password field to clarify behavior with and without username

Reviewed Changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated 6 comments.

File Description
zh_CN/access-control/authz/redis.md Added Dashboard configuration section and username field documentation for Chinese authorization docs
zh_CN/access-control/authn/redis.md Added username field documentation and restructured password field description for Chinese authentication docs
en_US/access-control/authz/redis.md Added username field documentation and reorganized configuration instructions for English authorization docs
en_US/access-control/authn/redis.md Added username field documentation with Redis ACL details for English authentication docs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Meggielqk Meggielqk marked this pull request as ready for review October 28, 2025 01:42
@Meggielqk Meggielqk requested a review from zmstone October 28, 2025 01:43
@zmstone zmstone merged commit cd34452 into release-5.8 Oct 30, 2025
6 checks passed
@zmstone zmstone deleted the 20251022-redis-auhn-update branch October 30, 2025 10:31
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.

3 participants