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

Deprecate HttpClientHTTPConduitFactory value of *.http-conduit-factory options #1633

Closed
ppalaga opened this issue Dec 1, 2024 · 1 comment · Fixed by #1637
Closed

Deprecate HttpClientHTTPConduitFactory value of *.http-conduit-factory options #1633

ppalaga opened this issue Dec 1, 2024 · 1 comment · Fixed by #1637

Comments

@ppalaga
Copy link
Contributor

ppalaga commented Dec 1, 2024

HttpClientHTTPConduit never gained any real traction within Quarkus CXF. When CXF started using it as a default, we were forced to introduce our own default (URLConnectionHTTPConduitFactory) to avoid bugs like #992, CXF-8885, CXF-8951, CXF-8946, CXF-8903 and possibly others.
Now that we have VertxHttpClientHTTPConduit that we can support very well on Quarkus, there are no more reasons for us to spend resources on HttpClientHTTPConduit.

We should mark it as deprecated in our documentation, add some warnings on startup for the case that somebody is still using it

@ppalaga
Copy link
Contributor Author

ppalaga commented Dec 2, 2024

Follow up for the complete removal of the option: #1638

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.

1 participant