oscerd commented on code in PR #29221:
URL: https://github.com/apache/camel/pull/29221#discussion_r4479037736
##########
components/camel-platform-http-vertx/src/main/java/org/apache/camel/component/platform/http/vertx/VertxPlatformHttpServer.java:
##########
@@ -254,6 +254,14 @@ protected void startServer() throws Exception {
}
},
executor).toCompletableFuture().join();
+
+ Throwable cause = failure.get();
+ if (cause != null) {
+ throw new RuntimeCamelException(
+ "Failed to start Vert.x HttpServer on " +
configuration.getBindHost() + ":" + configuration.getBindPort()
+ + ", reason: " +
cause.getMessage(),
Review Comment:
Good point, fixed in a7bcac07d3de: the message now appends the throwable
itself (`reason: java.net.BindException: Address already in use`), so the class
is named even when the message is null.
_Claude Code on behalf of Andrea Cosentino_
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
