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

[HTTP] Catch and ignore exception from an external server #89452

Merged
merged 3 commits into from
Jul 26, 2023
Merged

[HTTP] Catch and ignore exception from an external server #89452

merged 3 commits into from
Jul 26, 2023

Conversation

ManickaP
Copy link
Member

@ManickaP ManickaP commented Jul 25, 2023 •
edited
Loading

Specific error codes pulled from kusto:

AzureDevOpsTests
| where TestName has "GetAsync_SetAutomaticDecompression_ContentDecompressed_Deflate"
| summarize max(RunCompleted) by Message
Message max_RunCompleted
System.Net.Http.HttpRequestException : Response status code does not indicate success: 504 (Gateway Time-out). 2023-07-25T13:06:00.503Z
System.Net.Http.HttpRequestException : Response status code does not indicate success: 502 (Bad Gateway). 2023-07-25T12:09:21.28Z
System.Net.Http.HttpRequestException : Resource temporarily unavailable (httpbin.org:80)
---- System.Net.Sockets.SocketException : Resource temporarily unavailable		 2023-07-04T12:47:25.46Z
System.Net.Http.HttpRequestException : Resource temporarily unavailable (httpbin.org:443)
---- System.Net.Sockets.SocketException : Resource temporarily unavailable		 2023-07-01T12:42:15.28Z

Fixes #87864

@ghost ghost assigned ManickaP Jul 25, 2023
@ManickaP ManickaP requested a review from a team July 25, 2023 17:05
@ghost
Copy link

ghost commented Jul 25, 2023

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

Issue Details

Fixes #87864

Author: ManickaP
Assignees: -
Labels:

area-System.Net.Http
Milestone: -

@ManickaP
Copy link
Member Author

/azp list

@azure-pipelines
Copy link

CI/CD Pipelines for this repository:

@ManickaP
Copy link
Member Author

/azp run runtime-libraries-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ManickaP @MihaZupan
Update src/libraries/Common/tests/System/Net/Http/HttpClientHandlerTe…
89e87ee 
…st.RemoteServer.cs

Co-authored-by: Miha Zupan <mihazupan.zupan1@gmail.com>
@ManickaP
Copy link
Member Author

/azp run runtime-libraries-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@ManickaP
Copy link
Member Author

/azp run runtime-libraries-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@ManickaP
Copy link
Member Author

/azp list

@azure-pipelines
Copy link

CI/CD Pipelines for this repository:

@ManickaP
Copy link
Member Author

/azp run runtime-libraries-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@build-analysis build-analysis bot mentioned this pull request Jul 26, 2023
Closed
@ManickaP
Copy link
Member Author

Failures unrelated.

@ManickaP ManickaP merged commit f937797 into dotnet:main Jul 26, 2023
100 of 111 checks passed
@ManickaP ManickaP deleted the mapichov/http-outerloop-fix branch July 26, 2023 14:29
@karelz karelz added this to the 8.0.0 milestone Aug 1, 2023
@rzikm rzikm mentioned this pull request Aug 1, 2023
Closed
@ghost ghost locked as resolved and limited conversation to collaborators Aug 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@CarnaViire CarnaViire CarnaViire approved these changes

@MihaZupan MihaZupan MihaZupan approved these changes

Assignees

@ManickaP ManickaP

Labels
area-System.Net.Http
Projects
None yet
Milestone
8.0.0
Development

Successfully merging this pull request may close these issues.

[TestFailure] GetAsync_SetAutomaticDecompression_ContentDecompressed_Deflate (Gateway Timeout)
4 participants
@ManickaP @CarnaViire @MihaZupan @karelz