Skip to content
This repository has been archived by the owner on Jul 9, 2024. It is now read-only.

RetryHandler should add more information on failures #221

Closed
andrueastman opened this issue Mar 22, 2024 · 1 comment
Closed

RetryHandler should add more information on failures #221

andrueastman opened this issue Mar 22, 2024 · 1 comment
Labels
enhancement New feature or request Standard GitHub label Issue caused by core project dependency modules or library

Comments

@andrueastman
Copy link
Member

andrueastman commented Mar 22, 2024

Sourced from microsoftgraph/msgraph-beta-sdk-dotnet#816

The default retry handler simply throws an InvalidOperationException exception when it reaches the maximum retry count. The error message is simply set to: «Too many retries performed.» and the inner exception simply states: «More than {retryCount} retries encountered while sending the request.»
There is no way to know why retries have been performed. What were the error codes and error messages?

The retry handler should return an AggregateException (or a specific exception like TooManyRetriesException) with one exception per retry specifying the HTTP status and the error code returned by the server.

@andrueastman
Copy link
Member Author

This was resolved via #223

@github-project-automation github-project-automation bot moved this from Todo 📃 to Done ✔️ in Kiota Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request Standard GitHub label Issue caused by core project dependency modules or library
Projects
Archived in project
Development

No branches or pull requests

1 participant