PlatinumTeam / MarbleBot

Bot for the discord server
0 stars 0 forks source link

Rate limiting #1

Open JeffProgrammer opened 7 years ago

JeffProgrammer commented 7 years ago

Discord has a rate limit. We will probably have to queue up chat messages for X number of seconds and then blast them off into the discord server. We don't want to submit a message for every server.

hPerks commented 7 years ago

Why not just handle the rate limits when they happen? When a bot is rate limited, the API gives you back a number of milliseconds to wait before sending again; you can just put things on a queue then, instead of doing it all the time. Seems needlessly slow for people who might be doing quick running commentary or something (cough tourney matches).

https://github.com/hammerandchisel/discord-api-docs/blob/master/docs/topics/Rate_Limits.md

JeffProgrammer commented 7 years ago

my concern is that the discord system is intelligent enough to be able to determine who keeps hitting the rate limit and apply even higher ones to the point where it is limited badly. Even if they don't do that know, who knows when/if. And tbh i'd rather not find out.

hPerks commented 7 years ago

Good point - maybe we should use the rate limit headers they give you normally then to make sure things don't go too fast. For one thing, I don't want to start displaying multiple messages at a time and get this alignment problem, if we don't have to:

image