-
Notifications
You must be signed in to change notification settings - Fork 62
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
Can't make a pure client #281
Comments
hmmm if we need to support both, we need to split server and client module . And of course common module. |
A split would certainly be the clearest but also the most elaborate approach. |
@NKame 's issue of disabling the HTTP server looks like it may possibly be related to a Quarkus regression: quarkusio/quarkus#19370 |
Well, it seems to come and go, the basis is that as soon as there is some mandatory dependency towars quarkus-vertx-http, by whatever module, the bootstrapping of HTTP server will kick in. And depending on the version, asking to disable HTTP either is almost correct (still the sequence is 90% done) or fails badly. |
The Also, the effort involved in splitting into server and client modules may be relatively involved ... so, have you by any chance tried excluding <dependency>
<groupId>io.quarkiverse.cxf</groupId>
<artifactId>quarkus-cxf</artifactId>
<exclusions>
<exclusion>
<groupId>io.quarkus</groupId>
<artifactId>quarkus-vertx-http</artifactId>
</exclusion>
</exclusions>
</dependency> |
As I explained it's quite simple: depending on the versions, it's working or not, I suppose most of the users don't try to use it because of this reliability problem and put quarkus.http.port=0 to not be bothered with it. As for excluding the dependency I've done several tests: most of the times it breaks during bootstrapping. |
Thanks @NKame, I was hoping there may be some less complex workaround to address the issue. I did some digging, and it seems the |
Well, it's a very low priority issue. I submitted it just in case some other people were wondering wether this has to be expected or not. |
I created quarkusio/quarkus#20119 to fix |
quarkusio/quarkus#20119 was merged and will be part of Quarkus 2.3.0 and possibly also 2.2.4. |
FWIW, with Quarkus 2.3.0.Final |
I think we should add a note to the docs. |
Good point @famod , reopening this issue, will close once the docs have been updated. Thanks for the fix! |
The |
Context: building a command line batch in Quarkus, using quarkus-cxf for the SOAP client part.
Expectation: the batch is not a server, so it should not listen to any socket, and even less run a HTTP server.
Problem: the HTTP server is launched, setting quarkus.http.host-enabled=false fails with java.lang.IllegalArgumentException: Must configure at least one of http, https or unix domain socket
Root cause: quarkus-cxf has a mandatory dependency towards quarkus-vertx-http
The text was updated successfully, but these errors were encountered: