----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73151/ -----------------------------------------------------------
(Updated Jan. 28, 2021, 3:11 p.m.) Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, Nikhil Bonte, and Sarath Subramanian. Changes ------- Addressed Review comments from Sarath Bugs: ATLAS-4107 https://issues.apache.org/jira/browse/ATLAS-4107 Repository: atlas Description ------- Atlas not picking the ldap bind password from the correct jceks file. There are two jceks file in Atlas configured from CM, ldap bind password should be retrieved from hadoop.security.credential.provider.path instead cert.stores.credential.provider.path. Diffs (updated) ----- intg/src/main/java/org/apache/atlas/ApplicationProperties.java e662c8fae intg/src/main/java/org/apache/atlas/security/SecurityProperties.java 2147cd17c intg/src/main/java/org/apache/atlas/security/SecurityUtil.java 082c6378c Diff: https://reviews.apache.org/r/73151/diff/2/ Changes: https://reviews.apache.org/r/73151/diff/1-2/ Testing ------- Tested on normal cluster with password stored in jceks file and TLS cluster with password stored in jceks file in Atlas. Thanks, Nixon Rodrigues