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

AsyncStreamingClient does not respect exponential backoff ratelimit for 429 #1982

Closed
TrustyJAID opened this issue Oct 12, 2022 · 2 comments
Closed
Labels
Bug This is regarding a bug with the library

Comments

@TrustyJAID
Copy link

After setting up the AsyncStreamingClient for v2 I noticed a massive number of 429 error codes, thousands within 24 hours. I did some digging to figure out what is going on and haven't been able to spot the exact cause. What I did notice though is that twitters documentation says that you must use exponential backoff if the error code is 429 starting at 60 seconds but the code only does exponential backoff starting at the minimum 60 seconds when the error code is 420 which I have yet to see. This inevitably leads to the stream constantly attempting to reconnect too early increasing the time before it can reconnect the stream again due to too many connections leading to downtime on the stream.

@deboomce
Copy link

This relates to issue #1986

@Harmon758 Harmon758 added the Bug This is regarding a bug with the library label Oct 22, 2022
@Harmon758
Copy link
Member

Thanks for the report.
I noticed this recently, but I hadn't gotten around to committing a fix for it yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is regarding a bug with the library
Projects
None yet
Development

No branches or pull requests

3 participants