Add Validation for Multipart Form Type using resteasy reactive

[abuelgasimsaadeldin]

Description

Previously when using the quarkus-resteasy-multipart we could add validation to dto requests such as @NotNull and @notblank however after migrating to resteasy reactive we are unable to add validations.

Implementation ideas

Add validations similar to previous quarkus-resteasy-multipart.

[quarkus-bot]

/cc @FroMage, @geoand, @stuartwdouglas

[geoand]

This actually already works!

You just need to annotate the @Multipart method parameter with @Valid as well, see #26459 for an example

[abuelgasimsaadeldin]

Hi @geoand, yes if I'm not mistaken the javax.validation library before came by default with the previous quarkus-resteasy-multipart however with the new reseasy reactive it does not come with the @Valid annotation by default so may I know the dependency that is required to be added to my pom.xml to be able to use this library? Thanks!

[geoand]

Yeah, to get @Valid and friends just add quarkus-hibernate-validator to your dependencies.

[abuelgasimsaadeldin]

Thanks!

[geoand]

🙏🏼

[abuelgasimsaadeldin]

Hi @geoand I have another simple inquiry if you do not mind regarding the request dto, previously before migrating to resteasy reactive I used to have the @FormParam annotation providing the KEY value for the request and String variable for the variable initialization, however I noticed that now the KEY value is using the variable name if that makes sense.

For example:

    @FormParam("destination_country")
    @PartType(MediaType.TEXT_PLAIN)
    private String destinationCountry;

and in Swagger instead of of setting the KEY to "destination_country" (which is required) it instead uses "desinationCountry" which gives out errors so I had to initialize my variables with String destination_country which I believe is not common practice and instead should be using camelCase, your input in this is highly appreciated. Thanks.

Example swagger UI when setting @FormParam("destination_country") and String destinationCountry:

Expected swagger UI:

[geoand]

Can you please open a new issue for this?

[abuelgasimsaadeldin]

@geoand yes, sure

[abuelgasimsaadeldin]

@geoand Done created, I was not sure whether to submit it as a bug or question so I apologize if it I had mistakenly grouped it wrongly. But here it is: #26461

[geoand]

Thanks