vertx-web icon indicating copy to clipboard operation
vertx-web copied to clipboard

MultipartForm sets TRANSFER-ENCODING chunked

Open DemonicTutor opened this issue 3 years ago • 8 comments

Version

Vertx 4.3.4

Context

https://netty.io/news/2022/09/08/4-1-81-Final.html

Connection related headers in HTTP/2 frames are now rejected, in compliance with the specification

Reproducer

https://github.com/DemonicTutor/vertx-434-issues https://github.com/DemonicTutor/vertx-434-issues/blob/main/src/test/java/com/noenv/vertx434issues/MultipartFormTest.java

Steps to reproduce

  1. WebClient.sendMultipartForm

Extra

  • OpenJdk 19

DemonicTutor avatar Oct 14 '22 14:10 DemonicTutor

@DemonicTutor mutipart form is handled by vertx-core as it delegates to netty to do the parsing

pmlopes avatar Oct 21 '22 13:10 pmlopes

im talking about the client part... WebClient ueses https://github.com/vert-x3/vertx-web/blob/master/vertx-web-client/src/main/java/io/vertx/ext/web/client/impl/MultipartFormUpload.java#L69-L72

and the DefaultFullHttpRequest Http_1_1 sets the header

yeah VertX is using Netty but specifically Http 1 implementation

DemonicTutor avatar Oct 21 '22 16:10 DemonicTutor