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

Add a regression test for double Content-Length header in HttpWebRequest #102989

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

MihaZupan
Copy link
Member

@MihaZupan MihaZupan commented Jun 3, 2024

Replaces #102986

Adds a test for the impacted scenario.

@MihaZupan MihaZupan added this to the 9.0.0 milestone Jun 3, 2024
@MihaZupan MihaZupan requested a review from ManickaP June 3, 2024 12:22
@MihaZupan MihaZupan self-assigned this Jun 3, 2024
Copy link
Contributor

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

@MihaZupan
Copy link
Member Author

/azp run runtime-libraries-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@liveans liveans left a comment

Choose a reason for hiding this comment

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

LGTM, can you add a test for this case?

@ManickaP
Copy link
Member

ManickaP commented Jun 4, 2024

Should we at least keep the test from this PR?

@MihaZupan MihaZupan reopened this Jun 4, 2024
@MihaZupan MihaZupan changed the title Fix double Content-Length header in HttpWebRequest Add a regression test for double Content-Length header in HttpWebRequest Jun 4, 2024
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.

4 participants