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



It would probably be a good idea to include "ssl" or "tls" in this 
configuration tag, as it's not obvious it refers to SSL configuration: 
"ranger.enabled.Protocols.defaultvalue"

- Colm O hEigeartaigh


On May 2, 2017, 6:42 a.m., bhavik patel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58915/
> -----------------------------------------------------------
> 
> (Updated May 2, 2017, 6:42 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Madhan 
> Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, 
> and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-1500
>     https://issues.apache.org/jira/browse/RANGER-1500
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Add support to disable/exclude weaker SSL protocols like TLSv1, TLSv1.1 for 
> ranger.
> 
> 
> Diffs
> -----
> 
>   
> embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/EmbeddedServer.java
>  aef59bd 
>   security-admin/src/main/resources/conf.dist/ranger-admin-default-site.xml 
> fb6d844 
>   security-admin/src/main/resources/conf.dist/ranger-admin-site.xml a4c36e0 
> 
> 
> Diff: https://reviews.apache.org/r/58915/diff/1/
> 
> 
> Testing
> -------
> 
> 1. Verifeid Ranger Admin & Ranger KMS on SSL enabled environment with 
> excluding different protocols.
> 
> 
> Thanks,
> 
> bhavik patel
> 
>

Reply via email to