ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Nitin Galave <nitin.gal...@gmail.com>
Subject Review Request 70711: RANGER-2446 : Suggestion - Include security zone details as part of admin audit for policy update
Date Fri, 24 May 2019 06:25:01 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70711/
-----------------------------------------------------------

Review request for ranger, Gautam Borad, Mehul Parikh, Pradeep Agrawal, and Velmurugan Periasamy.


Bugs: RANGER-2446
    https://issues.apache.org/jira/browse/RANGER-2446


Repository: ranger


Description
-------

If a policy which is part of security zone is edited, it will be good to include the zonename
also along with policyid and policyname in the popup where admin audit details are displayed


Diffs
-----

  security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java 0276367

  security-admin/src/main/webapp/scripts/views/policymanager/ServiceLayout.js 767f278 
  security-admin/src/main/webapp/scripts/views/reports/PlugableServiceDiffDetail.js cc9ee8d

  security-admin/src/main/webapp/scripts/views/security_zone/ZoneCreate.js e0fab23 
  security-admin/src/main/webapp/templates/reports/PlugableServicePolicyDeleteDiff_tmpl.html
04aa6dc 
  security-admin/src/main/webapp/templates/reports/PlugableServicePolicyDiff_tmpl.html dbc519a

  security-admin/src/main/webapp/templates/reports/PlugableServicePolicyUpdateDiff_tmpl.html
e150c32 


Diff: https://reviews.apache.org/r/70711/diff/1/


Testing
-------

Verified that " Zone name" get sdisplayed on audit>>admin tab:
1. When performed CRUD operation on Zoned-resource based policies.
2. When performed CRUD operation on Zoned-Tag based policies.


Thanks,

Nitin Galave


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message