You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2022/06/27 21:15:26 UTC

[GitHub] [ozone] kerneltime commented on pull request #3552: HDDS-6949. Add audit of change bucket ACL S3 API.

kerneltime commented on PR #3552:
URL: https://github.com/apache/ozone/pull/3552#issuecomment-1167908774

   > Thanks @kerneltime for updating the patch. I think moving the log statement to `putAcl` method would be more consistent, since error is also logged there, but this is good enough for fixing the bug.
   > 
   > Also, please try to avoid force-push next time.
   
   There is a separate effort on going to not log customer generated messages into s3g log. The audit log should be at the higher level and this should be the right place.


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org