yyqdbngt opened a new pull request, #3350:
URL: https://github.com/apache/rocketmq-dashboard/pull/3350

   ## Summary
   
   Marks the persisted plain-ACL user row's `accessKey` and `secretKey` as 
`@ToString.Exclude` and adds `RmqAclUserTest` locking the contract.
   
   Coverage:
   - `toString` keeps the non-secret fields (`username`, `admin`) while 
omitting both the credential field names and their values;
   - `@Data` equality/hashCode still cover the credential keys, so a rotated 
secret is not equal.
   
   ## Why
   
   The entity is logged during ACL user lifecycle operations; exposing AK/SK 
through `toString` is a credential leak vector (same class of issue as the 
cloud credential row).
   
   ## Testing
   
   `mvn -B test -Dtest=RmqAclUserTest` — 2/2 pass; checkstyle (validate) clean.


-- 
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]

Reply via email to