Skip to content
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.

Sign up for GitHub

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

Jump to bottom

MSAL retry policy #3561

Closed
1 of 2 tasks
bgavrilMS opened this issue Aug 5, 2022 · 2 comments · Fixed by #3733
Closed
1 of 2 tasks

MSAL retry policy #3561

bgavrilMS opened this issue Aug 5, 2022 · 2 comments · Fixed by #3733
Assignees
@bgavrilMS
Labels
Feature Request ICM This issue has a corresponding ICM, either for our team or another.
Milestone
4.48.0

Comments

@bgavrilMS
Copy link
Member

bgavrilMS commented Aug 5, 2022 •
edited
Loading

- [ ] Fix existing retry policy to retry once on TaskCancelledException and other HTTP errors like Socket exception etc.

In particular, we should advise on obeying the Retry-After headers.
@bgavrilMS bgavrilMS added this to the 4.47.0 milestone Aug 5, 2022
@bgavrilMS bgavrilMS added Feature Request availability ICM This issue has a corresponding ICM, either for our team or another. labels Aug 5, 2022
@bgavrilMS bgavrilMS changed the title Publish guidance on retry policies MSAL retry policy Aug 5, 2022
@bgavrilMS bgavrilMS mentioned this issue Aug 5, 2022
Closed
@pmaytak pmaytak mentioned this issue Aug 9, 2022
Closed
7 tasks
@trwalke trwalke self-assigned this Aug 15, 2022
@trwalke trwalke moved this from Estimated/Committed to In Progress in MSAL Customer Trust / QM Aug 15, 2022
@trwalke trwalke moved this from In Progress to Blocked/Waiting for reply in MSAL Customer Trust / QM Aug 23, 2022
@bgavrilMS bgavrilMS moved this from Blocked/Waiting for reply to Estimated/Committed in MSAL Customer Trust / QM Aug 26, 2022
@bgavrilMS bgavrilMS moved this from Estimated/Committed to Blocked/Waiting for reply in MSAL Customer Trust / QM Aug 26, 2022
@pmaytak pmaytak mentioned this issue Sep 1, 2022
Open
@bgavrilMS bgavrilMS moved this from Blocked/Waiting for reply to Estimated/Committed in MSAL Customer Trust / QM Sep 6, 2022
@bgavrilMS
Copy link
Member Author

bgavrilMS commented Sep 6, 2022 •
edited
Loading

Not sure if this is blocked by anything...

@bgavrilMS bgavrilMS self-assigned this Sep 7, 2022
@bgavrilMS bgavrilMS moved this from Estimated/Committed to In Progress in MSAL Customer Trust / QM Sep 7, 2022
@bgavrilMS
Copy link
Member Author

Related: #3661

@gladjohn gladjohn modified the milestones: 4.47.0, 4.48.0 Sep 17, 2022
@bgavrilMS bgavrilMS moved this from In Progress to Estimated/Committed in MSAL Customer Trust / QM Oct 3, 2022
@bgavrilMS bgavrilMS mentioned this issue Oct 4, 2022
Merged
1 task
@bgavrilMS bgavrilMS moved this from Estimated/Committed to Waiting for Code Review in MSAL Customer Trust / QM Oct 5, 2022
@bgavrilMS bgavrilMS moved this from Waiting for Code Review to Fixed in MSAL Customer Trust / QM Oct 5, 2022
@bgavrilMS bgavrilMS moved this from Fixed to Waiting for Code Review in MSAL Customer Trust / QM Oct 5, 2022
@bgavrilMS bgavrilMS moved this from Waiting for Code Review to Fixed in MSAL Customer Trust / QM Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bgavrilMS bgavrilMS

Labels
Feature Request ICM This issue has a corresponding ICM, either for our team or another.
Projects
MSAL Customer Trust / QM
Archived in project
Milestone
4.48.0
Development

Successfully merging a pull request may close this issue.

Add a test demonstrating retry policy AzureAD/microsoft-authentication-library-for-dotnet
3 participants
@bgavrilMS @trwalke @gladjohn