-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add a request pipeline and configuration for retries #469
Conversation
# Conflicts: # graph-http/src/client.rs
Hey @DevLazio this looks great. I apologize for not getting back to this sooner. I have been busy as of late. This is my fault for not saying anything sooner, but version 2.0.0 is being released and you will probably have to deal with merge conflicts when merging master. This is awesome though. I appreciate the hard work on this. |
@DevLazio Hopefully the merge conflicts are not too bad. Everything looks good to me and ready to merge once the conflicts have been resolved. |
Working on the merge conflicts while @DevLazio is away to help out and try to get this merged. |
This PR makes |
…reeise#469)"" This reverts commit 37e4eef.
* Make `EnvironmentCredential::username_password_env` public * Revert "Add a request pipeline and configuration for retries (sreeise#469)" This reverts commit 804467c. * Update public_client_application.rs * Update graph.rs * Update authorization_code_credential.rs * Update authorization_code_credential.rs * create 'From<AuthorizationCodeCredential> for PublicClientApplication<AuthorizationCodeCredential>' * Revert "Revert "Add a request pipeline and configuration for retries (sreeise#469)"" This reverts commit 37e4eef. * Move tower service setup to RequestHandler. Closes sreeise#491 (sreeise#494) * Move tower service setup to RequestHandler. Closes sreeise#491 --------- Co-authored-by: Sean Reeise <[email protected]>
Closes #421