Replace deprecated API in HTTP modules

quarkus-qe / quarkus-test-suite

16 stars 34 forks source link

Replace deprecated API in HTTP modules #2198

Closed mocenas closed 1 week ago

mocenas commented 1 week ago

Summary

Replace deprecated APIs in HTTP modules.

@MultipartForm Can be removed without replacement in certain conditions (that we meet).

Please select the relevant options.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Dependency update
[X] Refactoring
[ ] Backport
[ ] New scenario (non-breaking change which adds functionality)
[ ] This change requires a documentation update
[x] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[ ] Methods and classes used in PR scenarios are meaningful
[X] Commits are well encapsulated and follow the best practices

mocenas commented 1 week ago

run tests

rsvoboda commented 1 week ago

in certain conditions (that we meet) ... where are the conditions specified? Are they visible to users?

mocenas commented 1 week ago

in certain conditions (that we meet) ... where are the conditions specified? Are they visible to users?

They are specified in the deprecation message for the @MultipartForm it states this annotation is not required anymore: you can use BeanParam or just omit it entirely, as long as your container class holds any annotated fields with RestForm, RestCookie, RestHeader, RestPath, RestMatrix, RestQuery or their JAX-RS equivalents.

Yes this is visible to users.

Sorry if I wrote my original description too mysteriously.