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

Preserve parsed & raw header value ordering #67227

Merged
merged 1 commit into from
Mar 29, 2022

Conversation

MihaZupan
Copy link
Member

Fixes #63831

Just a matter of swapping the enumeration order.

@MihaZupan MihaZupan added this to the 7.0.0 milestone Mar 28, 2022
@ghost ghost assigned MihaZupan Mar 28, 2022
@ghost
Copy link

ghost commented Mar 28, 2022

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #63831

Just a matter of swapping the enumeration order.

Author: MihaZupan
Assignees: -
Labels:

area-System.Net.Http

Milestone: 7.0.0

@MihaZupan MihaZupan requested a review from a team March 28, 2022 10:54
@MihaZupan
Copy link
Member Author

MihaZupan commented Mar 29, 2022

Test failures are unrelated: #66694 #66970

Copy link
Member

@ManickaP ManickaP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@MihaZupan MihaZupan merged commit 69f60b3 into dotnet:main Mar 29, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTTP Headers are enumerated in the wrong order when added via a mix of parsed and unparsed APIs
2 participants