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

Non-compliant Content-Type header for multipart media type #26290

Closed
ijakus opened this issue Dec 16, 2020 · 0 comments
Closed

Non-compliant Content-Type header for multipart media type #26290

ijakus opened this issue Dec 16, 2020 · 0 comments
Assignees
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: bug A general bug

Comments

@ijakus
Copy link

ijakus commented Dec 16, 2020

Affects: 5.x


Two months ago similar issue has been submitted but has already been fixed and released so I'm opening a new one.

I also came across a server that is not robust enough and fails when charset is used as a parameter inside Content-Type header for multipart media type. As specified by RFC7578, only boundary is a required parameter and there are no optional parameters.

I'm submitting an issue for FormHttpMessageConverter. Same logic applied as a fix for #25885 can probably be used in this situation also.

Edit:
I know there is a workaround where one can use setMultipartCharset() to prevent charset from showing in header. In my opinion it would be better if Content-Type header was aligned with specification and it worked in same fashion as MultipartWriterSupport now works.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Dec 17, 2020
@poutsma poutsma self-assigned this Dec 18, 2020
@poutsma poutsma closed this as completed in ce1ae2f Jan 8, 2021
@poutsma poutsma added in: web Issues in web modules (web, webmvc, webflux, websocket) type: bug A general bug and removed for: team-attention status: waiting-for-triage An issue we've not yet triaged or decided on labels Jan 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: bug A general bug
Projects
None yet
Development

No branches or pull requests

3 participants