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

extensions-support httpclient: The BasicAuthCache alias may not be needed anymore when adopting http client >= 5.4 #5704

Closed
aldettinger opened this issue Jan 31, 2024 · 1 comment · Fixed by #6188
Assignees
Labels
Milestone

Comments

@aldettinger
Copy link
Contributor

Describe the feature here

Since https://issues.apache.org/jira/browse/HTTPCLIENT-2070, the BasicAuthCache is not using reflection anymore.
So we should probably be able to remove this native alias: https://github.com/apache/camel-quarkus/blob/main/extensions-support/httpclient/runtime/src/main/java/org/apache/camel/quarkus/support/httpclient/graalvm/BasicAuthCacheAlias.java

Some context could be found in #3079

@aldettinger aldettinger added the enhancement New feature or request label Jan 31, 2024
@jamesnetherton
Copy link
Contributor

The problem is that extensions-support-httpclient is still supporting components where httpclient 4.x is on the classpath.

Maybe we need a separate support extension for 4.x & 5.x as was hinted at in #4703.

@jamesnetherton jamesnetherton self-assigned this 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
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants