Resteasy reactive - missing @Form annotation

[kucharzyk]

Describe the bug

In classic resteasy there is @Form annotation which allow extract form parameters to separate class. It's especially usefull with qute. Endpoints that saves large forms can have many fields. Too many to put them to method signature.

In reasteasy reactive it's missing.

https://docs.jboss.org/resteasy/docs/6.0.0.Final/userguide/html/_Form.html

Expected behavior

Reasteasy reactive should have @Form annotation like reasteasy classic.

Actual behavior

@Form annotation is missing in reasteasy reactive

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

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

No response

Additional information

No response

[quarkus-bot]

/cc @FroMage, @geoand, @stuartwdouglas

[kucharzyk]

/cc @geoand

[geoand]

Sounds like a useful feature, thanks for reporting!

[fwippe]

There is already an annotation @BeanParam that IMHO could be used for that. Alas, it doesn't yet support @FormParams. I am going to create a PR to remedy that.

[geoand]

@fwippe you are very correct! I had forgot about @BeanParam since I have never used it myself...

Am pretty sure that it does support @FormParam already. What makes you think it does not?
There is also a test that uses it.

[fwippe]

@fwippe you are very correct! I had forgot about @BeanParam since I have never used it myself...

Am pretty sure that it does support @FormParam already. What makes you think it does not? There is also a test that uses it.

Because I thought so myself and tried to use it in a company project 🙂 Looking at io.quarkus.jaxrs.client.reactive.deployment.JaxrsClientReactiveProcessor#addBeanParamData there are cases for BeanParam, QueryParam, CookieParam, HeaderParam and PathParam, but no FormParam ☹

[geoand]

Ah, you mean the client, I didn't look at the client code.

If you want to provide a PR to fix that @fwippe, that would be much apprecited.

Thanks

[geoand]

Let's close this one as Won't fix, since @BeanParam seems to cover what is requested.

Sorry for having you open an issue @kucharzyk, I should have remember about @BeanParam.

And thanks @fwippe for stepping up :)

[fwippe]

For the record: #23099 will enable @FormParams for @BeanParams for the Reactive Rest Client