search

aj-r / RiotNet

A .NET/C# client for the Riot Games API
MIT License
23 stars 10 forks source link

Add proactive rate limiting #14

Closed aj-r closed 7 years ago

aj-r commented 7 years ago

Try to prevent requests from being sent if we think they will go over the rate limit.

aj-r commented 7 years ago

Mostly done - just need to see if we can use the X-App-Rate-Limit-Count header

aj-r commented 7 years ago

Decided not to use X-App-Rate-Limit-Count because it's too unreliable for concurrent requests, and its usefulness is very limited (only impacts the situation where you restart the process shortly after sending several requests, and even then it would just save you a couple of seconds). The rest of proactive rate limiting is working well.