----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62757/#review186998 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariLocalAuthenticationProvider.java Lines 43 (patched) <https://reviews.apache.org/r/62757/#comment263950> This was renamed from AmbariLocalUserProvider to be more consisten with other authentication providers. It was also moved from `org.apache.ambari.server.security.authorization` to `org.apache.ambari.server.security.authentication` ambari-server/src/test/java/org/apache/ambari/server/security/authentication/AmbariLocalAuthenticationProviderTest.java Lines 40 (patched) <https://reviews.apache.org/r/62757/#comment263951> This was renamed from `AmbariLocalUserProviderTest`. The generic authentication provider tests were moved to `AbstractAuthenticationProviderTes` to provide a way to test all relevant authentication providers so that they behave in a similar way. - Robert Levas On Oct. 3, 2017, 4:23 p.m., Robert Levas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62757/ > ----------------------------------------------------------- > > (Updated Oct. 3, 2017, 4:23 p.m.) > > > Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene > Chekanskiy, Jonathan Hurley, Laszlo Puskas, Nate Cole, Robert Nettleton, and > Sebastian Toader. > > > Bugs: AMBARI-21220 > https://issues.apache.org/jira/browse/AMBARI-21220 > > > Repository: ambari > > > Description > ------- > > Update Local Authentication process to work with improved user management > facility > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java > 3099bc0857 > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java > 21ab757e1c > > ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AccountDisabledException.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariAuthenticationEventHandlerImpl.java > 2a894374d1 > > ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariAuthenticationProvider.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariLocalAuthenticationProvider.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserProvider.java > 2a2e397439 > > ambari-server/src/main/java/org/apache/ambari/server/security/authorization/Users.java > 2dedc9e6ca > > ambari-server/src/test/java/org/apache/ambari/server/security/authentication/AbstractAuthenticationProviderTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/security/authentication/AmbariLocalAuthenticationProviderTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationProviderDisableUserTest.java > fea7fb978e > > ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariLocalUserProviderTest.java > fb4ebf93e4 > > > Diff: https://reviews.apache.org/r/62757/diff/1/ > > > Testing > ------- > > Manually tested > > Local test results: PASSED > > > Thanks, > > Robert Levas > >
