[GitHub] [accumulo] ctubbsii opened a new pull request #1827: Fix unneeded audit message about denial

[GitBox <gi...@apache.org>]

ctubbsii opened a new pull request #1827:
URL: https://github.com/apache/accumulo/pull/1827


   Reverse the order of permissions checks to check if the current user is
   asking about themselves. This prevents an unnecessary audit log message
   showing a denial when the user does not also have system privileges.


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

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

[GitHub] [accumulo] ctubbsii merged pull request #1827: Fix unneeded audit message about denial

[GitBox <gi...@apache.org>]

ctubbsii merged pull request #1827:
URL: https://github.com/apache/accumulo/pull/1827


   


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

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

[GitHub] [accumulo] ctubbsii commented on pull request #1827: Fix unneeded audit message about denial

[GitBox <gi...@apache.org>]

ctubbsii commented on pull request #1827:
URL: https://github.com/apache/accumulo/pull/1827#issuecomment-741063843


   This was reverted, as explained in 1fa84ae2ea7b40081a2c1a94a560cda2f91b5f36
   I will need to address this a different way.


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

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