Skip to content
This repository has been archived by the owner on Feb 4, 2021. It is now read-only.

Update interval 0 triggers endless 429 requests #529

Open
stoically opened this issue Feb 18, 2020 · 4 comments
Open

Update interval 0 triggers endless 429 requests #529

stoically opened this issue Feb 18, 2020 · 4 comments
Labels

Comments

@stoically
Copy link

stoically commented Feb 18, 2020

Thanks for the Add-on!

System Info

  • Firefox Version: 73.0
  • Extension Version: 3.11.1

Steps to reproduce

  • Hit the twitch api rate limit
  • Set update interval to 0 in the preferences
  • Watch the endless stream of 429 requests in the background network inspector
  • (also hogs a fair amount of the CPU due to the endless requests)

Workaround: Set update interval to 4294967295

@freaktechnik
Copy link
Owner

Solution: don't set it to 0 if you didn't expect it to keep updating forever and always.

@stoically
Copy link
Author

Well, the description of that preference reads Interval of the addon checking wether a stream is live in seconds. When 0, the states aren't automatically updated - so I was expecting that 0 disables the interval; or was that a misunderstanding?

@freaktechnik
Copy link
Owner

Oh, you then indeed aren't.

@stoically
Copy link
Author

It also seems that the update interval isn't respected when the Add-on initializes, e.g. on Firefox start it always refreshes the streams.

On related note: would be cool to have an option to refresh the streams when clicking the browser action (would spare one click on the refresh icon when disabling automatic refresh is preferred)

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

No branches or pull requests

2 participants