Skip to content
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

Eviter le spam automatiquement en cas de problème répétitif avec les crons #65

Open
targos opened this issue Oct 20, 2022 · 14 comments

Comments

@targos
Copy link
Member

targos commented Oct 20, 2022

Il y a un tas de solutions différentes possibles. Une des plus simples à implémenter je pense est un petit compteur (in memory) pour chaque cron. Si le run a fail 2 ou 3 fois d'affilée avec la même erreur, on log l'erreur une dernière fois en indiquant dans le message que les erreurs suivantes seront ignorées.
Dès que le run ne fail plus (ou fail avec une erreur différente), le compteur est reset.

@rayandfz
Copy link
Contributor

Hey, si je comprend bien :

  • Faire une variable compteur qui s'incrémente jusqu'à n=3 erreurs
  • Vérifier à chaque envoie d'un message d'erreur la valeur de n, si n = 3, on envoie le message en disant qu'on ignore les suivants, si n > 3 on ignore
  • Si le CRON réussis, on remet à zéro le compteur si il est diffèrent de 0

@targos
Copy link
Member Author

targos commented Oct 20, 2022

c'est ça!

@rayandfz
Copy link
Contributor

Je m'en occupe

@Xstoudi
Copy link
Member

Xstoudi commented Oct 20, 2022

Ca serait intéressant de simplement update le premier message de log pour ajouter un truc genre (x3), (x4), etc. si le suivant est identique.

@rayandfz
Copy link
Contributor

Je vais mettre à jour en mettant ta proposition dans la PR

@Xstoudi
Copy link
Member

Xstoudi commented Oct 20, 2022

Faut pas se presser, il faut laisser aux autres le temps de donner leur avis. 😁

@rayandfz
Copy link
Contributor

Yes, je vais un peu vite :D

@targos
Copy link
Member Author

targos commented Oct 21, 2022

ça m'a l'air d'une bonne idée. Comment on pourrait faire ça? En retenant l'id du message après l'avoir posté ou qch dans le genre?

@Xstoudi
Copy link
Member

Xstoudi commented Oct 21, 2022

Yes, on le retient pendant le temps T ou on veut pas se faire double-notifier.

@Xstoudi
Copy link
Member

Xstoudi commented Oct 21, 2022

(ou son hash, peu importe)

@targos
Copy link
Member Author

targos commented Oct 21, 2022

Je demande principalement parce que je ne connais pas trop l'API de Discord. Je ne sais pas ce qu'il faut pour pouvoir mettre à jour un message existant.

@Xstoudi
Copy link
Member

Xstoudi commented Oct 21, 2022

https://discord.js.org/#/docs/discord.js/main/class/Message?scrollTo=edit

J'pense qu'on peut s'en tirer avec ça.

@rayandfz
Copy link
Contributor

On peut le développer ? :)

@Xstoudi
Copy link
Member

Xstoudi commented Oct 25, 2022

Je pense que tu peux proposer un truc yes !

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

No branches or pull requests

3 participants