adoroszlai opened a new pull request, #9524:
URL: https://github.com/apache/ozone/pull/9524

   ## What changes were proposed in this pull request?
   
   `EndpointBase` has methods to simplify auditing failures in subclasses, but:
   - these are used in only some places
   - `SUCCESS` case is not covered
   
   This PR adds the missing `auditReadSuccess` and `auditWriteSuccess`, and 
changes all calls to use the simplified methods, except 
`BucketEndpoint#multiDelete`, which needs to add extra params for the message.
   
   Also:
   - Return `AuditMessage.Builder` from the helper methods to allow further 
customization of the message (add `perf`, etc.), thus reduce duplication.
   - Move `getAuditParameters()` call into `auditMessageBaseBuilder`, renamed 
to `auditMessageFor`.
   
   https://issues.apache.org/jira/browse/HDDS-14204
   
   ## How was this patch tested?
   
   CI:
   https://github.com/adoroszlai/ozone/actions/runs/20336484273 (in progress)
   https://github.com/adoroszlai/ozone/actions/runs/20334289695 (mostly OK)


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to