jamesnetherton commented on code in PR #17528:
URL: https://github.com/apache/camel/pull/17528#discussion_r2010306477
##########
components/camel-platform-http-vertx/src/main/java/org/apache/camel/component/platform/http/vertx/DefaultHttpRequestBodyHandler.java:
##########
@@ -27,21 +27,27 @@
import static
org.apache.camel.component.platform.http.vertx.VertxPlatformHttpSupport.isMultiPartFormData;
/**
- * Default {@link HttpRequestBodyHandler} that will read to read the entire
HTTP request body into memory.
+ * Default {@link HttpRequestBodyHandler} that will read the entire HTTP
request body into memory if useBodyHandler is
+ * enabled.
*/
class DefaultHttpRequestBodyHandler extends HttpRequestBodyHandler {
- DefaultHttpRequestBodyHandler(Handler<RoutingContext> delegate) {
+ final boolean useBodyHandler;
+
+ DefaultHttpRequestBodyHandler(Handler<RoutingContext> delegate, boolean
useBodyHandler) {
Review Comment:
I am confused as to what's making the request hang. It's maybe related to
the setup and configuration of `VertxPlatformHttpServer` used in the tests.
If I run a similar test on Camel Quarkus, it seems to work. Likewise If I
create just a plain Vert.x Server -> Client test (E.g no Camel), it also works.
--
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-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
