-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63044/#review188313
-----------------------------------------------------------


Ship it!




Ship It!

- Nate Cole


On Oct. 16, 2017, 5:02 p.m., Robert Levas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63044/
> -----------------------------------------------------------
> 
> (Updated Oct. 16, 2017, 5:02 p.m.)
> 
> 
> Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene 
> Chekanskiy, Jonathan Hurley, Laszlo Puskas, Myroslav Papirkovskyy, Nate Cole, 
> Robert Nettleton, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-21217
>     https://issues.apache.org/jira/browse/AMBARI-21217
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Update JWT Authentication process to work with improved user management 
> facility
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/api/AmbariErrorHandler.java
>  a57effcd62 
>   
> ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java
>  62e8b864f5 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
>  0d24ef21e7 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariAuthenticationEventHandlerImpl.java
>  4cfce2a373 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariAuthenticationFilter.java
>  b3bc4c31fe 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariAuthenticationProvider.java
>  3d20cb9ec2 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariBasicAuthenticationFilter.java
>  3667012ae4 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariJWTAuthenticationFilter.java
>  3d35578524 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariLocalAuthenticationProvider.java
>  dcdf4710d4 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/UserNotFoundException.java
>  0f2fbb6477 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/jwt/AmbariJwtAuthenticationProvider.java
>  PRE-CREATION 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/jwt/JwtAuthenticationToken.java
>  PRE-CREATION 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authentication/kerberos/AmbariKerberosAuthenticationFilter.java
>  23fa1715b1 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariPamAuthenticationProvider.java
>  0823729905 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariUserAuthentication.java
>  9445882b62 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariUserAuthorizationFilter.java
>  8fbd81614a 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authorization/jwt/JwtAuthentication.java
>  7b21ce667d 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authorization/jwt/JwtAuthenticationFilter.java
>  f42df6cb24 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/authorization/jwt/JwtAuthenticationProperties.java
>  cb456fa6b4 
>   
> ambari-server/src/main/java/org/apache/ambari/server/security/ldap/AmbariLdapDataPopulator.java
>  32dd6dc4b4 
>   ambari-server/src/main/resources/webapp/WEB-INF/spring-security.xml 
> 6650f67c22 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/SecurityHelperImplTest.java
>  4d6d5a9e73 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/authentication/AbstractAuthenticationProviderTest.java
>  96b4883fb2 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/authentication/AmbariJWTAuthenticationFilterTest.java
>  14c103223e 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/authentication/AmbariLocalAuthenticationProviderTest.java
>  d445c07e9a 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/authentication/jwt/AmbariJwtAuthenticationFilterTest.java
>  PRE-CREATION 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariPamAuthenticationProviderTest.java
>  4e080b1617 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AuthorizationHelperTest.java
>  41816aa5c8 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/authorization/jwt/JwtAuthenticationFilterTest.java
>  47df030990 
>   
> ambari-server/src/test/java/org/apache/ambari/server/security/authorization/jwt/JwtAuthenticationPropertiesTest.java
>  144d90a712 
> 
> 
> Diff: https://reviews.apache.org/r/63044/diff/1/
> 
> 
> Testing
> -------
> 
> Manually tested
> 
> # Local test results
> ```
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 28:50 min
> [INFO] Finished at: 2017-10-16T16:53:23-04:00
> [INFO] Final Memory: 99M/1929M
> [INFO] 
> ------------------------------------------------------------------------
> ```
> 
> # Jenkins test result: PENDING
> 
> 
> Thanks,
> 
> Robert Levas
> 
>

Reply via email to