Skip to content

Conversation

@TropicolX
Copy link
Contributor

@TropicolX TropicolX commented Nov 8, 2025

Clarified the usage of the ACL SETUSER command and its effects on user properties. Added details on password storage and validation tests.


Note

Refines the auth stage docs to clarify ACL SETUSER for the default user, add SHA-256 password storage details, and update test steps and expected RESP outputs.

  • Docs (auth-05-uv9):
    • Clarify ACL SETUSER for the default user and capitalize section heading.
    • Add explicit examples showing command usage and RESP +OK response.
    • Introduce Password Storage: SHA-256 hashing, nopass flag removal, and example ACL GETUSER output.
    • Update Tests: precise redis-cli commands and expected RESP arrays before/after setting password; require +OK\r\n response.
    • Add note to store password hash as lowercase hex and rationale for hash-only storage.

Written by Cursor Bugbot for commit b489f51. This will update automatically on new commits. Configure here.

Clarified the usage of the `ACL SETUSER` command and its effects on user properties. Added details on password storage and validation tests.
@github-actions
Copy link

github-actions bot commented Nov 8, 2025

LLM Doc Lint

14 tests   14 ✅  0s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit c756cdb.

♻️ This comment has been updated with latest results.

Clarify command descriptions and server responses in the documentation.
@TropicolX TropicolX enabled auto-merge November 9, 2025 04:48
@TropicolX TropicolX merged commit 8c77e06 into main Nov 9, 2025
29 checks passed
@github-actions
Copy link

github-actions bot commented Nov 9, 2025

LLM Doc Lint

14 tests   14 ✅  0 ❌

1 suites

Results for commit 4d6528f.

This comment updates automatically on new commits.

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.

2 participants