Resteasy Reactive - Multipart does not accept text/plain bigger than 2048 bytes

[Postremus]

Describe the bug

I am currently migrating a small app from Resteasy Classic to Resteasy Reactive.

One of my resources uses multipart to accept form input from users.

2048 or less bytes work, the text field contains the input text.
If the users enters more than 2048 bytes, the text field is null.
No exception is thrown.

My resource:

@POST
    @Consumes(MediaType.MULTIPART_FORM_DATA)
    @Produces(MediaType.TEXT_PLAIN)
    public String hello(@MultipartForm SomeData data)

import org.jboss.resteasy.reactive.PartType;

import javax.ws.rs.FormParam;

public class SomeData {
    @FormParam("text")
    @PartType("text/plain")
    private String text;
}

Expected behavior

A text/plain field should not be arbitary limited to 2048 bytes.

It should work like with Resteasy Classic, with the only limit being the configured quarkus.http.limits.max-body-size.

Actual behavior

@PartType("text/plain") is left null if input text is longer than 2048 bytes.

To Reproduce

Steps to reproduce the behavior:

1. Download the reproducer: rr-large-multipart.zip
2. Run the GreetingResourceTest test class. testLargeText will fail.
3. Add another byte to the src/test/resources/2048.rj file.
4. Run the tests again. Now also test2048Bytes will fail.

Environment (please complete the following information):

Output of uname -a or ver

Linux martin 5.8.0-48-generic #54~20.04.1-Ubuntu SMP Sat Mar 20 13:40:25 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Output of java -version

openjdk version "15.0.1" 2020-10-20
OpenJDK Runtime Environment Zulu15.28+13-CA (build 15.0.1+8)
OpenJDK 64-Bit Server VM Zulu15.28+13-CA (build 15.0.1+8, mixed mode, sharing)

Quarkus version or git rev

1.13.1.Final

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
Maven home: /home/martin/.sdkman/candidates/maven/current
Java version: 15.0.1, vendor: Azul Systems, Inc., runtime: /home/martin/.sdkman/candidates/java/15.0.1-zulu
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "5.8.0-48-generic", arch: "amd64", family: "unix"

[quarkus-bot]

/cc @FroMage, @geoand, @stuartwdouglas

[geoand]

Thanks for reporting. I'll have a look tomorrow

[geoand]

@Postremus #16432 will address the issue.

However I have a question: Did you really mean to send the input as a form attribute or as a file? Because if you meant to do the latter, you are not doing it correctly :)

[Postremus]

@geoand Yes, that is intended :)

Back then when I created this app, this was the only way I know how to achieve this.
The text is input into a simple html form, which is then send as multipart/form-data to the application.

The app also has another multipart endpoint for accepting a complete file - which is working fine already in resteasy reactive :)

[geoand]

Understood, thanks for the info

[technical-debt-collector]

@geoand Kind of late to the party here, but this should probably be mentioned in the migration documentation as well.
quarkus.http.limits.max-form-attribute-size is only ever mentioned as a possible configuration, but hardly anything more from what I can find.
https://quarkus.io/guides/resteasy-reactive-migration#multipart-support

[geoand]

Feel free to open a PR with your suggestion.

Thanks

[technical-debt-collector]

Sure, will do