----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74521/#review225653 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java Line 247 (original), 271 (patched) <https://reviews.apache.org/r/74521/#comment314119> RangerDataset.admins was removed recently, since this can be determined by ADMIN privilege granted in RangerDataset.acl. Please review and update. - Madhan Neethiraj On July 24, 2023, 7:46 a.m., Prashant Satam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74521/ > ----------------------------------------------------------- > > (Updated July 24, 2023, 7:46 a.m.) > > > Review request for ranger, Madhan Neethiraj and Subhrat Chaudhary. > > > Bugs: RANGER-4324 > https://issues.apache.org/jira/browse/RANGER-4324 > > > Repository: ranger > > > Description > ------- > > Validating Access for RangerDataSet by checking if Requesting User is Present > in RangerGdsObjectACL i.e in users,groups,roles Also if Requesting User is > present in RangerDataSet.Admin section consisting of users,groups,roles of > RangerDataSet > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java > 817bc3d38 > > > Diff: https://reviews.apache.org/r/74521/diff/2/ > > > Testing > ------- > > Done Testing by Adding the Logged in User to Admins in RangerDataSet and also > adding the Logged in User to Acl of RangerDataSet > > > Thanks, > > Prashant Satam > >