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

Exclude http 4.x from http extension if possible. #4703

Closed
JiriOndrusek opened this issue Mar 27, 2023 · 0 comments · Fixed by #6188
Closed

Exclude http 4.x from http extension if possible. #4703

JiriOndrusek opened this issue Mar 27, 2023 · 0 comments · Fixed by #6188
Assignees
Milestone

Comments

@JiriOndrusek
Copy link
Contributor

Camel 4.x upgraded http client to version 5.x. Quarkus apache-http-client brings 4.x.
Package names are distinct, therefore there is no problem having both of them onclasspath.
Unfortunately it is highly error prone. So I suggest to exclude client 4.x from the http extension.
(I'm not sure if it is a simple task or not -> that's why I created this ticket)
Idea if other extensions depend on http, we might need to create http4 extension.

@JiriOndrusek JiriOndrusek self-assigned this Mar 27, 2023
@jamesnetherton jamesnetherton changed the title [camel-main] Exclude http 4.x from http extension if possible. Exclude http 4.x from http extension if possible. May 19, 2023
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jun 14, 2024
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jun 14, 2024
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jun 14, 2024
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jun 14, 2024
@github-actions github-actions bot added this to the 3.12.0 milestone Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants