requests / requests-oauthlib

OAuthlib support for Python-Requests!
https://requests-oauthlib.readthedocs.org/
ISC License
1.71k stars 421 forks source link

Update Client after refresh regardless of if `token_updater` is specified #487

Open hiporox opened 2 years ago

hiporox commented 2 years ago

It appears that if a token_updater is not specified, the client will not get updated. Instead of providing a default token_updater, we can instead move this client update call to happen after refresh regardless

Fixes https://github.com/requests/requests-oauthlib/issues/411

AdrienKnell commented 12 months ago

UP. Do we have any updates about this PR ? Can someone merge it ?

jtroussard commented 4 months ago

Pushing a small change to kick off the PR build checks and tests

coveralls commented 4 months ago

Coverage Status

coverage: 90.097% (-0.2%) from 90.31% when pulling 5e16ac3e90e7f4c5222b7aed81d1a0f8b24dfadb on hiporox:update_client_with_no_token_updater into b3decdb65872ba1f59b65f62406949306b673ddb on requests:master.

jtroussard commented 4 months ago

Looks like all the checks/unit tests have passed, is it possible to get/produce some evidence of integration or manual smoke/regression tests?