-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add vertx web client config #15192
Comments
Related to #15189 |
Definitely +1 for this idea. I don't see however how we could make these settings global as the
The
Perhaps we need some new SPI in Vertx where we can register the default |
At the moment the Vert.x Web Client is unmanaged. What about a facade like:
which would merge the options with some default ones and produce the web client? |
Yeah, that's a good idea |
+1 for this, we need to enable logs in Vert.x Web client and it seems this requires passing |
Do I understand it correctly that Clement's proposal should end up in the |
Yes, or in a new extension. |
We're using the vertx web client in RESTEasy Reactive, but there's no way to configure it. While we might add jaxrs-client-specific config to override global web client settings, there should be a way to configure the vertx web client globally, possibly in the vertx extension?
The text was updated successfully, but these errors were encountered: