[GitHub] [camel-quarkus] jamesnetherton opened a new issue, #4628: vertx-websocket producer path handling is broken for some scenarios

["jamesnetherton (via GitHub)" <gi...@apache.org>]

jamesnetherton opened a new issue, #4628:
URL: https://github.com/apache/camel-quarkus/issues/4628

   Producing to the internal Vert.x HTTP server using the shortened URI idiom `vertx-websocket:///test` is a bit broken. For example:
   
   ```
   from("vertx-websocket:///test")
       .log("Message ====> ${body}");
   
   from("direct:start")
       .to("vertx-websocket:///test");
   ```
   
   Results in:
   
   ```
   Caused by: java.util.concurrent.ExecutionException: io.netty.channel.AbstractChannel$AnnotatedConnectException: Connection refused: /[0.0.0.0:80](http://0.0.0.0/)
       at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
   	    at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
   	    at org.apache.camel.component.vertx.websocket.VertxWebsocketEndpoint.getWebSocket(VertxWebsocketEndpoint.java:167)
   	    at org.apache.camel.component.vertx.websocket.VertxWebsocketEndpoint.getWebSocket(VertxWebsocketEndpoint.java:173)
   	    at org.apache.camel.component.vertx.websocket.VertxWebsocketProducer.getConnectedPeers(VertxWebsocketProducer.java:128)
   	    at org.apache.camel.component.vertx.websocket.VertxWebsocketProducer.process(VertxWebsocketProducer.java:62)
   ```
   
   Also echoing back a response on the original WS channel works in a flawed way.
   
   This works as expected.
   
   ```
   from("vertx-websocket:///echo")
           .setBody(simple("Hello ${body}"))
           .to("vertx-websocket:///echo");
   ```
   
   But it also incorrectly works with `.to("vertx-websocket:///echostuff")`, `.to("vertx-websocket:///echo/more/stuff")`.
   
   We only have happy path tests in CQ, so this has gone unnoticed. I'll open a separate issue for improving the test coverage.


-- 
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] ppalaga closed issue #4628: vertx-websocket producer path handling is broken for some scenarios

["ppalaga (via GitHub)" <gi...@apache.org>]

ppalaga closed issue #4628: vertx-websocket producer path handling is broken for some scenarios
URL: https://github.com/apache/camel-quarkus/issues/4628


-- 
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