search

apache / camel-quarkus

Apache Camel Quarkus
https://camel.apache.org
Apache License 2.0
251 stars 186 forks source link

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

Closed aldettinger closed 3 weeks ago

aldettinger commented 5 months ago

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

jamesnetherton commented 5 months ago

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 https://github.com/apache/camel-quarkus/issues/4703.