> On April 11, 2022, 7:34 p.m., Ramesh Mani wrote: > > agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java > > Line 1125 (original), 1135 (patched) > > <https://reviews.apache.org/r/73941/diff/1/?file=2267345#file2267345line1135> > > > > should we consider auditPolicyId() also for this override?
Chained plugins, in the current form, are used to re-evaluate access policies across multiple component services. Perhaps considering audit policies with chained plugins can be addressed in another functional JIRA. This JIRA is mainly to consider improving log messages for help during debugging. - Abhay ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73941/#review224295 ----------------------------------------------------------- On April 11, 2022, 5:58 p.m., Abhay Kulkarni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73941/ > ----------------------------------------------------------- > > (Updated April 11, 2022, 5:58 p.m.) > > > Review request for ranger, Madhan Neethiraj and Ramesh Mani. > > > Bugs: RANGER-3705 > https://issues.apache.org/jira/browse/RANGER-3705 > > > Repository: ranger > > > Description > ------- > > Add/improve logging messages to help debugging potential issues. > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerServiceResourceMatcher.java > 7b02dd6e1 > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceTrie.java > 331d6371d > > agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java > 12fecd030 > > > Diff: https://reviews.apache.org/r/73941/diff/1/ > > > Testing > ------- > > Built clean and passed all unit tests. > > > Thanks, > > Abhay Kulkarni > >
