-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40837/
-----------------------------------------------------------
Review request for ranger, Alok Lal, Abhay Kulkarni, Madhan Neethiraj, Ramesh
Mani, and Velmurugan Periasamy.
Bugs: Ranger-755, Ranger-756 and Ranger-757
https://issues.apache.org/jira/browse/Ranger-755
https://issues.apache.org/jira/browse/Ranger-756
https://issues.apache.org/jira/browse/Ranger-757
Repository: ranger
Description
-------
"getopts" function in shell script reads the command line options one character
at a time. Hence -noauth is treated as -n, -o, -a, etc... which is the root
cause for Ranger-755.
For Ranger-756 & Ranger-757: According to the original design of ldap tool,
verification of authentication is intended to be performed only while
discovering all the usersync properties. Some of the conditions to disable auth
verification were not handled properly.
Diffs
-----
ugsync/ldapconfigchecktool/ldapconfigcheck/scripts/run.sh 787e216
ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/CommandLineOptions.java
790330f
Diff: https://reviews.apache.org/r/40837/diff/
Testing
-------
1. Tested various combinations on the command line arguments and verified the
desired results.
2. Also tested with both dev & QE setup to run different tests for regressions.
Thanks,
Sailaja Polavarapu