btlqql opened a new pull request, #2186: URL: https://github.com/apache/rocketmq-dashboard/pull/2186
## Summary - resolve Tencent ACL updates with the required role id - load the existing role only when a permission field is omitted - preserve each omitted permission instead of defaulting it to true - return the effective role and permission values after the update ## Root cause `UpdateAclUserDTO` models a partial update: `id` is required while `username`, `permRead`, and `permWrite` are optional. The Tencent adapter required the optional username and converted null permissions to true, rejecting valid id-only requests and unexpectedly enabling omitted permissions. Closes #2185. ## Validation - `mvn -Dtest=TencentAclServiceTest test` (2 tests passed) - `mvn -DskipTests checkstyle:check` (0 violations) - `git diff --check` - both changed paths checked against all open PRs targeting `rocketmq-studio` (no matches) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
