----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73451/#review223348 -----------------------------------------------------------
Ship it! Ship It! - Abhay Kulkarni On July 24, 2021, 8:41 a.m., Ramesh Mani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73451/ > ----------------------------------------------------------- > > (Updated July 24, 2021, 8:41 a.m.) > > > Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, > Mehul Parikh, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan > Periasamy. > > > Bugs: RANGER-3334 > https://issues.apache.org/jira/browse/RANGER-3334 > > > Repository: ranger > > > Description > ------- > > RANGER-3334:Enhance Ranger admin REST Client to use cookie for policy, tag > and role download > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/admin/client/RangerAdminRESTClient.java > e0d7a9b71 > > agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerAbstractContextEnricher.java > f838f842b > > agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerAdminTagRetriever.java > 7b49a9906 > > agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerAdminUserStoreRetriever.java > ed963363d > > agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerTagEnricher.java > a78b4842c > > agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerTagRetriever.java > 0f09b64d2 > > agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerUserStoreEnricher.java > 10616334c > > agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerUserStoreRetriever.java > 1addbc49d > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPluginContext.java > a8e76009e > > agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java > 82b048194 > > agents-common/src/main/java/org/apache/ranger/plugin/util/PolicyRefresher.java > dcba36338 > > agents-common/src/main/java/org/apache/ranger/plugin/util/RangerCommonConstants.java > 570eb6e36 > > agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTClient.java > 216b6b2a9 > > knox-agent/src/main/java/org/apache/ranger/admin/client/RangerAdminJersey2RESTClient.java > fa17f5d4b > > > Diff: https://reviews.apache.org/r/73451/diff/2/ > > > Testing > ------- > > - Testing done in local vm for policy, tag and role download. > - Access log in ranger admin will show the 401 for authentication call for > the first download and if there are no changes only 304 response will be > there. There won't be any 401 kerberos authentication call each time when > downloads are happening. > - Avoid instanciation of RangerAdminClient for each tag download, so that > cookie is preserved and used for subsequent calls. > > > Thanks, > > Ramesh Mani > >
