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.

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 token caching #98

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

Conversation

aaronjolson
Copy link

Add token caching and mechanism for refusing to send known bad tokens to provider. This allows us to keep the number of calls to a provider down in order to avoid incurring extra costs and/or bumping into rate limits.

Includes two class methods for being able to clear the instance's known good and known bad token caches.

@svintit
Copy link

svintit commented Jul 22, 2020

@aaronjolson If you want this resolved i'm maintaining a fork of this with some added extensibility. Feel free to open a PR here:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants