Request for advice on best way to implement rate limiter on requests-oauthlib

requests / requests-oauthlib

OAuthlib support for Python-Requests!

https://requests-oauthlib.readthedocs.org/

ISC License

1.72k stars 424 forks source link

Request for advice on best way to implement rate limiter on requests-oauthlib #436

Closed Seersucker closed 2 years ago

Seersucker commented 3 years ago

Hello Friend, Thanks for creating this great module! It has helped me solve the refresh token issue.

What is the preferred way for using requests-oathlib so it will play nice with a rate limiting API? Potential options are:

create a new Class and around requests like requests-oauthlib did for GET calls?
create a decorator like ratelimit
something else entirely?

Any direction is appreciated. Thanks, Andrew

JonathanHuot commented 2 years ago

Hi @Seersucker, is it still a requirement ? I don't see how requests-oauthlib can trigger a rate limiting API. It should only ask for a refresh token when required and should not bombard the authorization server.

If you have concrete use-cases to share, please re-open the issue.