-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
[Resilient HTTP apps] Add a paragraph describing how to disable retries for some HTTP methods #43630
[Resilient HTTP apps] Add a paragraph describing how to disable retries for some HTTP methods #43630
Conversation
Adds description how to disable retries for some HTTP methods
The API described in the PR will be released in dotnet/extensions 9.1.0 (https://github.com/dotnet/extensions). We want to merge the PR only after that release. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some suggestions but will leave it up to @IEvangelist to confirm
Co-authored-by: Darius Letterman <[email protected]>
Added the |
The package with the new API was published in .NET Extensions 9.1.0. But the API reference documentation (like the following one) doesn't still include the new API, and because of that the build is failing. @IEvangelist @RussKie do you know when the new API reference documentation will be published? Thank you. |
If the new NuGet package is live/public, we should just need to kick off a new .NET API docs build. I believe @gewarren will know best, so I'll defer to her. |
I can work on updating the API docs today. |
@gewarren Thank you. Do you need any information from my side? |
No, all good. The job is running and will be done in about 2.5 hours. |
APIs added in dotnet/dotnet-api-docs#10864. |
@gewarren Thank you very much for taking care of this PR. |
The PR adds description how to disable retries for some HTTP methods.
Fixes dotnet/extensions#5248
Internal previews