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

Multipart filename support RFC6266 without language #4943

Draft
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

zapodot
Copy link

@zapodot zapodot commented Dec 21, 2021

The RFC6266 does support having a language identifier as part of an encoded filename (content disposition param filename*), however the current Jersey Multipart parsing will not decode the filename if the language identifier is not present.

Example:

Content-Disposition: form-data; name=myfile; filename=myfile.pdf; filename*=utf-8''myfile.pdf

<content>

Currently when you bind this request to FormDataContentDisposition parameter the filename field value will be utf-8''myfile.pdf, but I would really expect it to be myfile.pdf

@jansupol
Copy link
Contributor

This looks like it makes sense, we do not need to have the charset a part of the header, it is already in the String actual charset.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants