Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Expand Resiliency Error Codes #70

Closed
kgidewall opened this issue Feb 13, 2017 · 1 comment
Closed

Expand Resiliency Error Codes #70

kgidewall opened this issue Feb 13, 2017 · 1 comment
Labels

Comments

@kgidewall
Copy link
Member

We currently invoke resiliency mode when the following conditions occur: network timeout or http error codes 500, 503, 504. After discussion, we have decided to expand the error code list to include ALL http error codes 500 - 599. Please change your logic to include all of those codes for Resiliency.

@rayluo
Copy link
Collaborator

rayluo commented Aug 22, 2021

Closing this due to 3 reasons:

  • At this point, ADAL Python no longer receives any new changes except bugfixes.
  • In the successor, the MSAL Python, we headed to a different direction anyway: errors and http errors could be retried in http layer (by a customized http client), and/or eventually being raised as exceptions.
  • Recently, MSAL Python also implemented a resiliency feature which focuses on not overloading the AAD service.

@rayluo rayluo closed this as completed Aug 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants