Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove OkHttp from BOM ? #34095

Closed
lasteris opened this issue Jun 16, 2023 · 7 comments
Closed

Remove OkHttp from BOM ? #34095

lasteris opened this issue Jun 16, 2023 · 7 comments
Labels
kind/question Further information is requested

Comments

@lasteris
Copy link
Contributor

lasteris commented Jun 16, 2023

Description

Just Recently, i've upgraded the bunch of services to Quarkus 3.1.1.Final.
It works perfectly itself. But after including minio i realized that we can use minio-client versions only lower than 8.2.3, because versions upper do not work due to okhttp < 4.8.1 which stricted by quarkus-bom.
After some researching i ve noticed this

#29520
and this
quarkiverse/quarkus-minio#220

and this

#30798

and even that
#30396

and this quarkiverse/quarkus-minio#144
which references to this #30480

but after reading all that i can not understand what is going on now. are we removing dependency ? why it still here ?

Maybe we can use this Issue to aggregate status about OkHttp ?

P.S. And maybe we can ask maintainers of minio-java to implement posibility of using not just okhttp, but any other http clients such as vert.x ? (or as example i will try PR with such kind implementation to minio-java by myself, but its just suggestion)

@lasteris lasteris added the area/housekeeping Issue type for generalized tasks not related to bugs or enhancements label Jun 16, 2023
@gastaldi gastaldi added kind/question Further information is requested and removed area/housekeeping Issue type for generalized tasks not related to bugs or enhancements labels Jun 16, 2023
@geoand
Copy link
Contributor

geoand commented Jun 19, 2023

We still need to some more work on this - this will likely happen after Quarkus 3.2 is released

@geoand geoand closed this as completed Jun 19, 2023
@lasteris
Copy link
Contributor Author

We still need to some more work on this - this will likely happen after Quarkus 3.2 is released

Thank you for the reply. I've just thought, maybe i could think and help with it somehow😀

@geoand
Copy link
Contributor

geoand commented Jun 19, 2023

There are further implications we need to work through, but thanks for volunteering

@languitar
Copy link
Contributor

Is the work on this tracked somewhere visibly. I was just hit by this via the meilisearch client that is dysfunctional with okhttp3 < 4.

@gsmet
Copy link
Member

gsmet commented May 21, 2024

Which version are you using?

We have removed the constraint in Quarkus 3.7: https://github.com/quarkusio/quarkus/wiki/Migration-Guide-3.7#okhttpokio-versions-not-enforced-anymore

@languitar
Copy link
Contributor

Oops, seems I've generated my extension for an older version of Quarkus than I thought. Sorry for the noise.

@gsmet
Copy link
Member

gsmet commented May 22, 2024

No worries :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/question Further information is requested
Projects
None yet
Development

No branches or pull requests

5 participants