You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Ramachandran Krishnan <ra...@gmail.com> on 2023/02/24 06:32:26 UTC

Review Request 74320: RANGER-4110: Upgrade to TLS to version 1.2 -Part2

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

Review request for ranger, Don Bosco Durai, Kirby Zhou, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Nikhil P, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, Subhrat Chaudhary, and Velmurugan Periasamy.


Bugs: RANGER-4110
    https://issues.apache.org/jira/browse/RANGER-4110


Repository: ranger


Description
-------

As part of  https://issues.apache.org/jira/browse/RANGER-3153 , we changed TLS to TLSv1.2

Still, the following classes are using TLS 
RangerRESTClient
RangerSslHelper
BaseAuditHandler
RemoteUnixLoginModule
DefaultSchemaRegistryClient
NiFiRegistryConnectionMgr

So, we need to change the above classes as well


Diffs
-----

  agents-audit/src/main/java/org/apache/ranger/audit/provider/BaseAuditHandler.java af09ca7b0 
  agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTClient.java 49e1281b7 
  agents-common/src/main/java/org/apache/ranger/plugin/util/RangerSslHelper.java 88c959c2a 
  plugin-nifi-registry/src/main/java/org/apache/ranger/services/nifi/registry/client/NiFiRegistryConnectionMgr.java 99eeced86 
  plugin-schema-registry/src/main/java/org/apache/ranger/services/schema/registry/client/connection/DefaultSchemaRegistryClient.java 8da7409d1 
  unixauthclient/src/main/java/org/apache/ranger/authentication/unix/jaas/RemoteUnixLoginModule.java 204398f7e 


Diff: https://reviews.apache.org/r/74320/diff/1/


Testing
-------


Thanks,

Ramachandran Krishnan


Re: Review Request 74320: RANGER-4110: Upgrade to TLS to version 1.2 -Part2

Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74320/#review225250
-----------------------------------------------------------


Ship it!




Ship It!

- Madhan Neethiraj


On Feb. 24, 2023, 6:32 a.m., Ramachandran Krishnan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/74320/
> -----------------------------------------------------------
> 
> (Updated Feb. 24, 2023, 6:32 a.m.)
> 
> 
> Review request for ranger, Don Bosco Durai, Kirby Zhou, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Nikhil P, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, Subhrat Chaudhary, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-4110
>     https://issues.apache.org/jira/browse/RANGER-4110
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> As part of  https://issues.apache.org/jira/browse/RANGER-3153 , we changed TLS to TLSv1.2
> 
> Still, the following classes are using TLS 
> RangerRESTClient
> RangerSslHelper
> BaseAuditHandler
> RemoteUnixLoginModule
> DefaultSchemaRegistryClient
> NiFiRegistryConnectionMgr
> 
> So, we need to change the above classes as well
> 
> 
> Diffs
> -----
> 
>   agents-audit/src/main/java/org/apache/ranger/audit/provider/BaseAuditHandler.java af09ca7b0 
>   agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTClient.java 49e1281b7 
>   agents-common/src/main/java/org/apache/ranger/plugin/util/RangerSslHelper.java 88c959c2a 
>   plugin-nifi-registry/src/main/java/org/apache/ranger/services/nifi/registry/client/NiFiRegistryConnectionMgr.java 99eeced86 
>   plugin-schema-registry/src/main/java/org/apache/ranger/services/schema/registry/client/connection/DefaultSchemaRegistryClient.java 8da7409d1 
>   unixauthclient/src/main/java/org/apache/ranger/authentication/unix/jaas/RemoteUnixLoginModule.java 204398f7e 
> 
> 
> Diff: https://reviews.apache.org/r/74320/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ramachandran Krishnan
> 
>