New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consolidate HTTP Interface client support for MultipartFile and @RequestPart
#31164
Consolidate HTTP Interface client support for MultipartFile and @RequestPart
#31164
Conversation
Signed-off-by: Dmitrii Bocharov <bdshadow@gmail.com>
spring-projectsgh-31139 Signed-off-by: Dmitrii Bocharov <bdshadow@gmail.com>
@RequestPart
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the pull request. Indeed, as we merge the two into one, we would no longer need MultipartFileArgumentResolver
and since it is not yet released, it can be removed, and likewise the test classes should also be merged into one.
Would you like to apply those changes?
@rstoyanchev yes, i will do it |
…tArgumentResolverTests Signed-off-by: Dmitrii Bocharov <bdshadow@gmail.com>
Also add test with Optional parameter for a RequestPart argument. See gh-31164 Signed-off-by: Dmitrii Bocharov <bdshadow@gmail.com>
This is now merged, thanks for the pull request. |
As described in the ticket, I've merged
MultipartFileArgumentResolver
into theRequestPartArgumentResolver
. I also think this is a better option since the javadoc for@RequestPart
explicitly mentions work with theMultipartFile.
However, I'm unsure if
MultipartFileArgumentResolver
must be removed now. Probably it can, since the release 6.1 isn't out yet, so it's present only in the pre-releases.