Skip to content
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

[Bug] Spring-web:SynchronossPartHttpMessageReader should only create temp directory when needed #2395

Closed
1 of 2 tasks
casionone opened this issue Jul 1, 2022 · 0 comments
Closed
1 of 2 tasks
Labels
bug Something isn't working

Comments

@casionone
Copy link
Contributor

casionone commented Jul 1, 2022

Search before asking

  • I searched the issues and found no similar issues.

Linkis Component

linkis-service-gateway

What happened + What you expected to happen

The same as spring-projects/spring-framework#27092

Some solutions can be shared by referring to this https://blog.csdn.net/ankeway/article/details/86747313
Spring-web has stopped using Synchronoss as default multiparty parser as of version 5.3, where we ship our own DefaultPartHttpMessageReader, which only creates temp directories when the size exceeds a defined limit.

It is necessary to evaluate whether to upgrade the spring-web version separately or upgrade the spring-boot version.
spring-boot 2.3.12.RELEASE is already the last version of 2.3.x


官方有这个bug的反馈,且已经修复。

spring-web 从 5.3 版开始,已经停止使用 Synchronoss 作为默认的多方解析器,在其中发布了自己的DefaultPartHttpMessageReader,它仅在大小超过定义的限制时创建临时目录

具体的处理方案可以参考

需要评估下是单独升级spring-web版本(spring-web 5.2.15.RELEASE 到5.2.16.RELEASE),还是升级spring-boot版本。spring-boot 2.3.12.RELEASE 已经是2.3.x的最后一个版本

Relevent platform

none

Reproduction script

none

Anything else

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!
@casionone casionone added the bug Something isn't working label Jul 1, 2022
@casionone casionone changed the title [Bug] SynchronossPartHttpMessageReader should only create temp directory when needed [Bug] Spring-web:SynchronossPartHttpMessageReader should only create temp directory when needed Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants