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

fix Docker entrypoint, rate limits and logs #69

Merged
merged 6 commits into from
Dec 31, 2024

Conversation

obscurerichard
Copy link
Member

@obscurerichard obscurerichard commented Dec 31, 2024

  • ** Fix Docker entrypoint **
  • Fend off rate limits, quiet noisy stravalib log

Until we can tackle #45 and #64 upgrading Stravalib to a 2.x version we need do do something to improve our rate limit handling. This is a brute force approach.

stravalib.attributes throws useless WARNING messages constantly. SHut it
up.

Add a hacky delay between requests to avoid hitting Strava read rate limits.

Until we upgrade to Stravalib 2.x this hopefully will tide us over.
@obscurerichard obscurerichard changed the title fix rate limits and logs fix Docker entrypoint, rate limits and logs Dec 31, 2024
@obscurerichard obscurerichard added the bug Something isn't working label Dec 31, 2024
@obscurerichard obscurerichard merged commit 8f11dba into master Dec 31, 2024
4 checks passed
@obscurerichard obscurerichard deleted the fix-rate-limits-and-logs branch December 31, 2024 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant