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

   ## Summary
   
   Marks the plain-access view's `accessKey` and `secretKey` as 
`@ToString.Exclude` and adds `PlainAccessConfigVOTest` locking the contract.
   
   Coverage:
   - `toString` keeps the non-secret fields (`whiteRemoteAddress`, 
`defaultTopicPerm`) while omitting both credential field names and values;
   - `@Data` equality/hashCode still cover the keys, so a rotated secret is not 
equal.
   
   ## Why
   
   The VO carries plaintext credentials on the ACL write/read paths and is 
logged during ACL operations; exposing AK/SK through `toString` is a credential 
leak vector (same class of issue as the credential/acl-user rows).
   
   ## Testing
   
   `mvn -B test -Dtest=PlainAccessConfigVOTest` — 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