----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60627/ -----------------------------------------------------------
(Updated July 4, 2017, 10:34 a.m.) Review request for atlas, Apoorv Naik, Ashutosh Mestry, Madhan Neethiraj, Sarath Subramanian, and Hemanth Yamijala. Changes ------- This patch includes setHadoopJWTCookie method in AtlasBaseClient, this method can be used to update the fresh hadoopJWT cookie when old cookie is expired instead creating the new object of AtlasClient. Tested quickStart script with this AtlasClient api methods. Bugs: ATLAS-1906 https://issues.apache.org/jira/browse/ATLAS-1906 Repository: atlas Description ------- This patch adds support to AtlasClient(Rest) to add Knox SSO Cookie as authentication header for Atlas. Currently Atlas Client supports UGI & Basic Auth, as part of this jira i will overload the Atlas Client constructor to support Knox SSO cookie by adding below constructor. AtlasClient(atlasURL, cookieName, CookieValue, path, domain); AtlasClient(atlasURL, Cookie); Diffs (updated) ----- client/src/main/java/org/apache/atlas/AtlasAdminClient.java b61b2bf client/src/main/java/org/apache/atlas/AtlasBaseClient.java 98da51e client/src/main/java/org/apache/atlas/AtlasClient.java ec482da client/src/main/java/org/apache/atlas/AtlasClientV2.java 7e287e7 common/src/main/java/org/apache/atlas/utils/AuthenticationUtil.java 09d8085 Diff: https://reviews.apache.org/r/60627/diff/2/ Changes: https://reviews.apache.org/r/60627/diff/1-2/ Testing ------- Unit test cases are running fine using mvn clean install. Ran Quick Start and Atlas admin script using the new Atlasclient Constructor. AltasClient(atlasUrl, new Cookie("hadoop-jwt" , "eyJhbGciOiJSUzI1NiJ9.eyJzdWIiOiJhZG1pbiIsImlzcyI6IktOT1hTU08iLCJleHAiOjE1MDE5NzUzNTV9.K7lTaXniXJtrdkXAG_k7MwhWpL363NPC75fA4hvY7ilNXCOr-_gVV-ZLndVxMGFmbSl-cLHv3ormDH2W2hvUQhUDvXg3QN5iQGlEoVDPjO6O1yoNnMBW0VgVhAbSimRQ5NTMgAF09gkTsPIfG8Qhu2kcvkKEMFOfcsDyrKn2cRM"); Thanks, Nixon Rodrigues