----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75366/#review227268 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java Lines 4647 (patched) <https://reviews.apache.org/r/75366/#comment315507> If ranger.plugin.audit.filters is not included in config when a service is created, Ranger admin will use the default value of the configuration specified in corresponding service-def. Setting to empty value here will result in the default value not be used. This would cause regression. Please verify. - Madhan Neethiraj On Feb. 24, 2025, 11:01 a.m., Dineshkumar Yadav wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/75366/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2025, 11:01 a.m.) > > > Review request for ranger, Kishor Gollapalliwar, Madhan Neethiraj, Mehul > Parikh, Pradeep Agrawal, and Vyom Tiwari. > > > Bugs: RANGER-5152 > https://issues.apache.org/jira/browse/RANGER-5152 > > > Repository: ranger > > > Description > ------- > > Service Create API adds the default "ranger.plugin.audit.filters" > configuration despite a provided blank value. > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > f7c521111 > > > Diff: https://reviews.apache.org/r/75366/diff/1/ > > > Testing > ------- > > Manual testing Done using below service create API > > /service/public/v2/api/service/ > > > Thanks, > > Dineshkumar Yadav > >
