[ 
https://issues.apache.org/jira/browse/RANGER-3993?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17790520#comment-17790520
 ] 

Vikas Kumar commented on RANGER-3993:
-------------------------------------

Hi [~bpatel] , Ranger KMS module has Azure keyvault dependency that internally 
uses okhttp. And that's the reason it is being included in the distro/kms.xml.

But I can not see "okhttp" inclusion for any other module, like if you check 
module xml files inside distro.

My point is, runtime dependencies should be included in distro xml files. Like 
we can simply include the compatible higher versions in distro/kms.xml and if 
required distro files for other modules can also be changed.

Example: 

for distro/kms.xml

<include>com.squareup.okhttp3:okhttp:4.11.0</include>

OkHttp4 is strictly backward compatible with okhtttp3.

Let's get it reviewed by other people as well. Thanks.

CC: [~mad...@apache.org] 

> Upgrade okhttp dependency 
> --------------------------
>
>                 Key: RANGER-3993
>                 URL: https://issues.apache.org/jira/browse/RANGER-3993
>             Project: Ranger
>          Issue Type: Bug
>          Components: Ranger
>            Reporter: Bhavik Patel
>            Assignee: Bhavik Patel
>            Priority: Major
>         Attachments: 0001-RANGER-3993-Upgrade-okhttp-dependency.patch
>
>
> Upgrade okhttp dependency 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to