Skip to content
This repository has been archived by the owner on May 11, 2022. It is now read-only.

Auth and Rate Limiting infrastructure #31 #34

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

thinkjrs
Copy link
Member

@thinkjrs thinkjrs commented Feb 7, 2021

When merged, this should add a new Auth and other classes to handle the following:

  • Initial lazy-loading of the token from a refresh_token parameter or environment variable.
  • Proper authentication token added to each request, using the requests auth built-in.
  • Decorator for base api call such that each request also monitors so that it
    1. Self-updates the token when expired
    2. Self-halts the client request if the limit has been reached

    I'd really like to get a queue going for requests on halting due to insufficient remaining api calls that continues after refresh!

@thinkjrs thinkjrs self-assigned this Feb 7, 2021
@github-actions github-actions bot added the client-dev Source code modification label Feb 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
client-dev Source code modification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant