Skip to content
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

WebClient fails when sending multipart formdata from a virtual thread #2696

Closed
bramdrenthe opened this issue Jan 7, 2025 · 2 comments · Fixed by #2700
Closed

WebClient fails when sending multipart formdata from a virtual thread #2696

bramdrenthe opened this issue Jan 7, 2025 · 2 comments · Fixed by #2700
Assignees
Labels
Milestone

Comments

@bramdrenthe
Copy link

Version

5.0.0 CR3

Context

Executing a webclient.sendMultipartForm with files > 250kb seems to fail when being called from a virtual thread.

Stacktrace

Exception: java.lang.IllegalArgumentException
io.vertx.ext.web.client.impl.MultipartFormUpload.pump:159
io.vertx.ext.web.client.impl.MultipartFormUpload$1.handleResume:69
io.netty.util.concurrent.AbstractEventExecutor.runTask:148
io.netty.util.concurrent.AbstractEventExecutor.safeExecute:141
io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks:521
io.netty.channel.SingleThreadIoEventLoop.run:160
io.netty.util.concurrent.SingleThreadEventExecutor$5.run:1123
io.netty.util.internal.ThreadExecutorMap$2.run:74
io.netty.util.concurrent.FastThreadLocalRunnable.run:30
java.lang.Thread.run:1583

Do you have a reproducer?

https://github.com/bramdrenthe/vertx-multipart-issue

Steps to reproduce

  1. Checkout above repository
  2. ./gradlew run
  3. Will start a local webserver at port 23456, and tries to send multipart formdata with 500kb file from both an eventloop verticle and a virtualthread verticle. Only the eventloop verticle will succeed.

Extra

Mac OS 15.2 // M2 Max
OpenJDK Runtime Environment (build 21.0.2+13-58)

@bramdrenthe bramdrenthe added the bug label Jan 7, 2025
@tsegismont tsegismont self-assigned this Jan 13, 2025
@tsegismont tsegismont modified the milestone: 5.0.0 Jan 13, 2025
@tsegismont
Copy link
Contributor

Looking into it

@tsegismont
Copy link
Contributor

Possible duplicate of #2685

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

2 participants