-
Notifications
You must be signed in to change notification settings - Fork 432
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
Update Microsoft.Extensions.Http.Resilience to 8.2 #2094
Conversation
This has a fix for dotnet/extensions#4770 which causes our EndToEnd tests to timeout if a service doesn't start in 100 seconds.
and maybe drop the retry from |
It is still needed because the project may return |
right, keep missing that. Then no try/catch? |
That's needed because when it returns |
…)" This reverts commit 2bf302b. There is a breaking change with Grpc clients in this version.
* Revert "Update Microsoft.Extensions.Http.Resilience to 8.2 (#2094)" This reverts commit 2bf302b. There is a breaking change with Grpc clients in this version. See dotnet/extensions#4924 * Set the EndToEnd test HttpClient timeout to infinite This allows the Http.Resilience handlers to handle the timeout correctly.
* Revert "Update Microsoft.Extensions.Http.Resilience to 8.2 (dotnet#2094)" This reverts commit 2bf302b. There is a breaking change with Grpc clients in this version. See dotnet/extensions#4924 * Set the EndToEnd test HttpClient timeout to infinite This allows the Http.Resilience handlers to handle the timeout correctly.
This has a fix for dotnet/extensions#4770 which causes our EndToEnd tests to timeout if a service doesn't start in 100 seconds.
We can update this to the official
8.2.0
when it ships.Microsoft Reviewers: Open in CodeFlow