-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45208/
-----------------------------------------------------------
(Updated March 29, 2016, 4:06 p.m.)
Review request for Ambari, Daniel Gergely, Robert Levas, and Sebastian Toader.
Changes
-------
- check base DN scope for ldap user/group sync
- added/updated tests
Bugs: AMBARI-15383
https://issues.apache.org/jira/browse/AMBARI-15383
Repository: ambari
Description (updated)
-------
Cleanup LDAP sync process:
- speedup sync with "--all" (do not check nested groups recursively...ambari
gather all of the groups first, and it's enough to just process them
sequentially)
- fixing issue: uppercase member attributes (in ambari.properties) don't work
well with the queries
- check that user/group DN (which is processed) is out of the scope or not
based on the base DN
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLdapUtils.java
ffebd45
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/Users.java
c2bc22f
ambari-server/src/main/java/org/apache/ambari/server/security/ldap/AmbariLdapDataPopulator.java
75df9cc
ambari-server/src/test/java/org/apache/ambari/server/security/AmbariLdapUtilsTest.java
b2778ae
ambari-server/src/test/java/org/apache/ambari/server/security/ldap/AmbariLdapDataPopulatorTest.java
ffff3ea
Diff: https://reviews.apache.org/r/45208/diff/
Testing
-------
Testing is in progress
Thanks,
Oliver Szabo
