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.

Sign up for GitHub

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

Jump to bottom

MockHttpServletResponse#setHeader does not remove header for null values #34466

Closed
github-actions bot opened this issue Feb 21, 2025 · 1 comment
Closed

MockHttpServletResponse#setHeader does not remove header for null values #34466

github-actions bot opened this issue Feb 21, 2025 · 1 comment
Assignees
@bclozel
Labels
in: test Issues in the test module type: backport An issue that is a backport of another issue to a maintenance branch type: bug A general bug
Milestone
6.1.18

Comments

@github-actions
Copy link
Contributor

Backport of gh-34464
@github-actions github-actions bot added in: test Issues in the test module type: backport An issue that is a backport of another issue to a maintenance branch type: bug A general bug labels Feb 21, 2025
@github-actions github-actions bot added this to the 6.1.18 milestone Feb 21, 2025
bclozel added a commit that referenced this issue Feb 21, 2025
@bclozel
Handle null values in MockHttpServletResponse#setHeader
b5c89c9 
Prior to this commit, `MockHttpServletResponse#setHeader` would not
remove the header entry when given a `null` value, as documented in the
Servlet API.
This commit ensures that this behavior is enforced.

Fixes gh-34466
@github-actions GitHub Actions
Copy link
Contributor Author

Fixed via b5c89c9

@bclozel bclozel self-assigned this Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bclozel bclozel

Labels
in: test Issues in the test module type: backport An issue that is a backport of another issue to a maintenance branch type: bug A general bug
Projects
None yet
Milestone
6.1.18
Development

No branches or pull requests
1 participant
@bclozel