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

fix(#2283): Prevent MultipartFormUpload PostRequestEncoder including transfer-encoding:chunked

Open zekronium opened this issue 1 year ago • 0 comments

Motivation:

Currently, MultipartFormUpload uses PostRequestEncoder which includes transfer-encoding headers which are illegal in HTTP2 and HTTP3 as per the RFC.

Currently it is a simple fix, maybe this should be improved from the netty side itself to prevent needing to rewrite/polyfill the entire PostRequestEncoder just for such a small thing.

Fixes #2283

Side note: When testing, I noticed that the MultipartFormUpload does not report any exceptions. If theres a failure in any of the handlers, the context is unaware. Is that intentional?

zekronium avatar Oct 23 '24 18:10 zekronium