Skip to content

Move the default should-retry logic into the default RetryOption#639

Merged
baywet merged 3 commits intomicrosoft:mainfrom
AndrewBennet:main
Feb 5, 2026
Merged

Move the default should-retry logic into the default RetryOption#639
baywet merged 3 commits intomicrosoft:mainfrom
AndrewBennet:main

Conversation

@AndrewBennet
Copy link
Contributor

@AndrewBennet AndrewBennet commented Feb 2, 2026

fixes #638

@AndrewBennet AndrewBennet requested a review from a team as a code owner February 2, 2026 18:08
@AndrewBennet
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Symplectic"

Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@github-project-automation github-project-automation bot moved this to In Progress 🚧 in Kiota Feb 3, 2026
Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

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

Thank you for making the changes!

@baywet baywet enabled auto-merge (squash) February 5, 2026 12:21
@baywet baywet merged commit 33d115c into microsoft:main Feb 5, 2026
7 checks passed
@github-project-automation github-project-automation bot moved this from In Progress 🚧 to Done ✔️ in Kiota Feb 5, 2026
This was referenced Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done ✔️

Development

Successfully merging this pull request may close these issues.

Cannot prevent RetryHandler from performing retries when response has HTTP status code of 429 or 503 or 504

2 participants