ctubbsii commented on PR #5483: URL: https://github.com/apache/accumulo/pull/5483#issuecomment-2811339996
@dlmarion I fixed the bug with the auditing, and tried to make some small changes to prevent similar bugs in the future. More work will be needed in the main branch to consolidate these classes, or refactor them so the checks done in the methods in these classes can be used more consistently. There are several operations that are currently not audited, because no method override exists in the AuditedSecurityOperation class. However, I did not try to fix all of those in this... I focused only on the one from batch scans. It looked like the code was already there for it in AuditedSecurityOperation, but it was not being called. -- 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]
