You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are also already rate limits for the different endpoints in the Client-Server API to prevent similar spam. I think it's good to act like a "good citizen" trying to prevent "spam" at the source.
I wonder if something in nio already takes care of rate limits blocking messages from being sent and retrying. There might not.
I'm assuming these are supposed to apply to the intervals of repeating reminders. What would be the purpose of the upper limit?
Note to potential implementers: remember cronjobs as well.
Observed behaviour
With the current implementation, the bot can be DoSed by setting up a recurring reminder as follows:
!remindroom every 1 second; now; spam
Proposed fix
Implement config variables for minimum and maximum timespans, e.g.
For reminders above and below the timespan limit, the bot should respond with an error.
The text was updated successfully, but these errors were encountered: