----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71225/#review217034 -----------------------------------------------------------
Ship it! Ship It! - Mehul Parikh On Aug. 1, 2019, 10:13 a.m., Nitin Galave wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71225/ > ----------------------------------------------------------- > > (Updated Aug. 1, 2019, 10:13 a.m.) > > > Review request for ranger, Gautam Borad, Madhan Neethiraj, Mehul Parikh, > Pradeep Agrawal, and Velmurugan Periasamy. > > > Bugs: RANGER-2517 > https://issues.apache.org/jira/browse/RANGER-2517 > > > Repository: ranger > > > Description > ------- > > UI patch for ranger policy model needs to be enhanced to support a feature > that implicitly denies access to the resource if the policy does not allow > access for requested operation. > > > Diffs > ----- > > security-admin/src/main/webapp/scripts/modules/globalize/message/en.js > 8078829 > security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js > a862dde > security-admin/src/main/webapp/scripts/views/policies/RangerPolicyRO.js > e9fcd7d > security-admin/src/main/webapp/styles/xa.css 3f39024 > > security-admin/src/main/webapp/templates/policies/RangerPolicyForm_tmpl.html > 7436829 > security-admin/src/main/webapp/templates/policies/RangerPolicyRO_tmpl.html > f8c6bce > > > Diff: https://reviews.apache.org/r/71225/diff/1/ > > > Testing > ------- > > 1. Verified "Deny All Other Accesses" toggle button on Policy form with > True/false values. > > 2. Performed basic CRUD operations. > > 3. If we set True value in "Deny All Other Accesses", Previous values gets > auto clear which was selected in Deny Conditions & Exclude from Deny > Conditions. > > > Thanks, > > Nitin Galave > >
