-
Notifications
You must be signed in to change notification settings - Fork 24
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
Spamschutz für "unnötige" Commands #142
Comments
Grundsätzlich stimme ich da zu, das sollten wir aber nicht jetzt integrieren sondern im Rahmen der Slash-Commands implementieren, damit wir nicht bei jeder Nachricht eine Datenbankabfrage, o.Ä. machen, sondern dediziert auf die Interaction Events horchen. Da können wir dann auch genau spezifizieren, welche Commands "von Natur aus" nicht spammen, weil die rEsponse ephemeral ist. |
dann mach mal REEEEEEEEEEEEEEEEEEEEEEEEEEE |
Ja, Mama |
ahhh ja fast vergessen. Auszeit (30 min) für den spammer fänd ich auch nicht verkehrt. |
Ist das wirklich nötig? Hatten wir jemals schon einen Spam? |
Nee bin auch der Meinung erstmal beobachten. Aber Info und Erleuchtung sind Recht frisch und haben relativ große embeds. Ist schon ein paar Mal vorgekommen, dass du beim scrollen nur embeds gesehen hast. Wird sich aber einpendeln, wenn die Commands nicht mehr neu und damit aufregend sind. |
Is das noch ein Problem? |
Eine Funktion, die übermäßiges benutzten von unnötigen Commands sperrt. Stattdessen wird eine Fehlermeldung, wie "ne du Hurensohn, ich hab grade kein bock und hör auf zu spammen" ausgegeben.
Der Channel bot-spam soll von dieser Funktion ignoriert werden.
Jedes Command soll zunächst als unnötig gelten. Ausnahmen von dieser Regelung soll für ein Command eingestellt werden können.
Implementationsvorschlag:
Die letzten 20? Commands werden mit user und timestamp gespeichert. (Datenbank ist optional, in mem reicht eigentlich) Wenn ein User mehr als 5? oder einem gewissen prozentsatz der letzten commands in einem gewissen zeitraum (30 min) abgeschickt hat, soll diese Funktion greifen.
The text was updated successfully, but these errors were encountered: