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 WinHttp SlowServerRespondsAfterDefaultReceiveTimeout_ThrowsHttpRequestException #104774

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

liveans
Copy link
Member

@liveans liveans commented Jul 12, 2024

Fixes #96481

Looks like issue is already fixed (Date response header was missing) with #104400, this PR makes it less error-prone.

@liveans liveans added NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels Jul 12, 2024
@liveans
Copy link
Member Author

liveans commented Jul 12, 2024

/azp run runtime-libraries-coreclr outerloop-windows

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liveans
Copy link
Member Author

liveans commented Jul 12, 2024

/azp run runtime-libraries-coreclr outerloop-windows

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liveans
Copy link
Member Author

liveans commented Jul 12, 2024

/azp run runtime-libraries-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liveans liveans force-pushed the investigate_96481 branch from 681f7eb to 671bc08 Compare July 12, 2024 12:35
@liveans liveans changed the title [NO REVIEW] Investigating WinHttp SlowServerRespondsAfterDefaultReceiveTimeout_ThrowsHttpRequestException Fix WinHttp SlowServerRespondsAfterDefaultReceiveTimeout_ThrowsHttpRequestException Jul 12, 2024
@liveans liveans removed NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels Jul 12, 2024
@liveans
Copy link
Member Author

liveans commented Jul 12, 2024

/azp run runtime-libraries-coreclr outerloop-windows

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liveans liveans marked this pull request as ready for review July 12, 2024 16:03
@liveans liveans requested a review from a team July 12, 2024 16:04
@liveans liveans merged commit a19fc08 into dotnet:main Jul 12, 2024
80 of 87 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 12, 2024
@karelz karelz added this to the 9.0.0 milestone Sep 3, 2024
@antonfirsov antonfirsov added the test-bug Problem in test source code (most likely) label Dec 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Net.Http test-bug Problem in test source code (most likely)
Projects
None yet
4 participants