Skip to content

Use unquoted value in assertion message in MediaType.checkParameters() #29461

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

Merged
merged 1 commit into from
Nov 10, 2022

Conversation

izeye
Copy link
Contributor

@izeye izeye commented Nov 10, 2022

Prior to 2a853ae, unquoted values were used there, but it seems that double quotes could be duplicated now. This PR tries to restore the previous behavior.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Nov 10, 2022
@jhoeller jhoeller added in: web Issues in web modules (web, webmvc, webflux, websocket) type: regression A bug that is also a regression and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Nov 10, 2022
@jhoeller jhoeller added this to the 6.0.0 milestone Nov 10, 2022
@jhoeller jhoeller self-assigned this Nov 10, 2022
@jhoeller jhoeller merged commit 4c2072d into spring-projects:main Nov 10, 2022
@jhoeller
Copy link
Contributor

Just realized that the original commit got backported to 5.3.x as well. Will fix it there with a cherry-pick.

@jhoeller jhoeller modified the milestones: 6.0.0, 5.3.24 Nov 10, 2022
@jhoeller jhoeller added type: task A general task and removed type: regression A bug that is also a regression labels Nov 10, 2022
@izeye izeye deleted the unquoted branch November 10, 2022 21:37
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: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants