-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58968/
-----------------------------------------------------------
(Updated May 3, 2017, 6:24 p.m.)
Review request for Ambari, Attila Doroszlai, Di Li, Robert Levas, and Tim
Thorpe.
Bugs: AMBARI-20909
https://issues.apache.org/jira/browse/AMBARI-20909
Repository: ambari
Description
-------
Create a local user "test" in Ambari
Create a system user "test" with different password
Trying to authenticate via pam in Ambari UI as user "test" throws Server Error
in Ambari UI, without any error in Ambari-server logs
Also the UI gets stalled and not even able to login as admin user unless the
browser cache is removed or Ambari UI is opened in a new browser page
Ambari doesn't allow creating users with same user name but different
types(Local/pam), We reach the pam authentication only when the local user
authentication failed due to non-existing userid or incorrect password, So if
local user exists do not attempt to authenticate via PAM, This lets Ambari to
avoid importing duplicate userid into the database
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariPamAuthenticationProvider.java
b3fb861
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariPamAuthenticationProviderTest.java
b7272c5
Diff: https://reviews.apache.org/r/58968/diff/2/
Changes: https://reviews.apache.org/r/58968/diff/1-2/
Testing
-------
Updated the related test cases
Thanks,
Anita Jebaraj
