You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/10/18 13:06:43 UTC
[GitHub] [camel-quarkus] ppalaga opened a new issue, #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
ppalaga opened a new issue, #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192
If `quarkus.cxf.path` is left unset, the Vert.x route for CXF is going to be mapped to a catch-all `/*` which in turn prevents other routes such as the ones for RESTeasy to deploy. So users should either set `quarkus.cxf.path` to some non-default value or they should make sure that they do not have any HTTP endpoints except for CXF.
We could perhaps also implement some checking of available `io.quarkus.vertx.http.deployment.RouteBuildItem` and fail if `/*` is not the only one.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] jamesnetherton commented on issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on issue #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192#issuecomment-1439916653
This would probably help eventually, where the management routes would be hosted on a separate port:
https://github.com/quarkusio/quarkus/pull/30506
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] ppalaga commented on issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on issue #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192#issuecomment-1496680577
I was able to come up with a PoC how to fix this properly in quarkus-cxf: https://github.com/quarkiverse/quarkus-cxf/pull/796
It is not perfect yet as some tests are failing. The idea is to register CXF only on paths that are really configured for it instead of `/*`.
[RestEasyTest.java](https://github.com/quarkiverse/quarkus-cxf/pull/796/files#diff-584d2c5351bc918e5fc71f5a6d7f64d76dcb18fd4b092fe109e6e482bef30881) is covering it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] ppalaga commented on issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on issue #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192#issuecomment-1488417293
> I've hit this issue while trying to deploy CQ + CXF on top of OpenShift. I've received such errros: `11:36:07,343 Can't find the request for http://10.129.2.151:8080/q/health/ready's Observer 11:36:07,346 Can't find the request for http://10.129.2.151:8080/q/health/ready's Observer` because when i don't configure `quarkus.cxf.path` then the `/q` endpoint is missing and the pod cannot run without liveness/readiness checks.
This use case should work after we upgraded to Quarkus 3.0.0.Beta1 where it is possible to let the health endpoints to be served on a separate network interface/port - see https://quarkus.io/version/main/guides/management-interface-reference
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] ppalaga closed issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga closed issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
URL: https://github.com/apache/camel-quarkus/issues/4192
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] jamesnetherton commented on issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on issue #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192#issuecomment-1464853528
Simler issues reported here:
https://stackoverflow.com/questions/75697754/apache-camel-on-quarkus-cxf-soap-problem-with-localhost8080
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] jamesnetherton commented on issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on issue #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192#issuecomment-1439918696
And it's not just RESTeasy endpoints that would be inaccessible. It's pretty much everything non-cxf. E.g static content, custom Vert.x routes etc.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] tmielke commented on issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "tmielke (via GitHub)" <gi...@apache.org>.
tmielke commented on issue #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192#issuecomment-1446379837
Is this something we may rather want to fix than just document? Docs are often easy to miss.
Not sure though how difficult a potential fix would be...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] llowinge commented on issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "llowinge (via GitHub)" <gi...@apache.org>.
llowinge commented on issue #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192#issuecomment-1439901367
I've hit this issue while trying to deploy CQ + CXF on top of OpenShift. I've received such errros: ```
11:36:07,343 Can't find the request for http://10.129.2.151:8080/q/health/ready's Observer
11:36:07,346 Can't find the request for http://10.129.2.151:8080/q/health/ready's Observer
``` because when i don't configure `quarkus.cxf.path` then the `/q` endpoint is missing and the pod cannot run without liveness/readiness checks.
@ppalaga The same problem will probably occur in plain Quarkus + OCP scenario, so i guess it should be not allowed from quarkus-cxf to catch-all with `/*`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] llowinge commented on issue #4192: Document the fact that the default value `/` of quarkus.cxf.path may cause RESTeasy endpoints to be inacessible
Posted by "llowinge (via GitHub)" <gi...@apache.org>.
llowinge commented on issue #4192:
URL: https://github.com/apache/camel-quarkus/issues/4192#issuecomment-1495517776
@ppalaga Revisiting and based on `By default, Quarkus exposes the management endpoints under /q on the main HTTP server. The same HTTP server provides the application endpoints and the management endpoints.` it means, that user which wants to use CXF + Quarkus + Cluster has to explicitely enable the `quarkus.management.enabled=true`. So indeed best way would be either:
* disable `"catch-all /*"`
or
* hardcode `quarkus.cxf.path` to some default value (eg. `"cxf"`)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org