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

fix(http): Make Content-Type header case insensitive #56541

Closed

Conversation

JeanMeche
Copy link
Member

Prior to this change, is the Content-Type passed to the FetchBackend was lowercase it was overwritten with the default one.

fixes #56539

@pullapprove pullapprove bot requested a review from AndrewKushnir June 21, 2024 10:34
@JeanMeche JeanMeche added action: review The PR is still awaiting reviews from at least one requested reviewer area: common/http labels Jun 21, 2024
@ngbot ngbot bot modified the milestone: Backlog Jun 21, 2024
Prior to this change, is the `Content-Type` passed to the `FetchBackend` was lowercase it was overwritten with the default one.

fixes angular#56539
@JeanMeche JeanMeche force-pushed the http-content-type-insensitive branch from 6d00d98 to 97f6761 Compare June 21, 2024 13:43
@AndrewKushnir AndrewKushnir added action: presubmit The PR is in need of a google3 presubmit target: minor This PR is targeted for the next minor release and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 22, 2024
@AndrewKushnir
Copy link
Contributor

Presubmit.

@AndrewKushnir AndrewKushnir added action: merge The PR is ready for merge by the caretaker and removed action: presubmit The PR is in need of a google3 presubmit labels Jun 23, 2024
@dylhunn
Copy link
Contributor

dylhunn commented Jun 25, 2024

This PR was merged into the repository by commit cc21989.

The changes were merged into the following branches: main

@dylhunn dylhunn closed this in cc21989 Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: common/http target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FetchBackend expects case sensitive 'Content-Type'
4 participants