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

AntiBot configuration cannot be changed with /authme reload #896

Closed
Twonox opened this issue Aug 10, 2016 · 6 comments
Closed

AntiBot configuration cannot be changed with /authme reload #896

Twonox opened this issue Aug 10, 2016 · 6 comments
Milestone

Comments

@Twonox
Copy link
Contributor

Twonox commented Aug 10, 2016

When we change value of enableAntiBot then reload authme, the value isn't applied. We must reboot the server (or reload?).

antiBotSensibility does not work, we can't change sensibility.

@sgdc3 sgdc3 added this to the 5.2 milestone Aug 10, 2016
@ljacqu ljacqu changed the title AntiBot configuration AntiBot configuration cannot be changed with /authme reload Aug 30, 2016
@ljacqu
Copy link
Member

ljacqu commented Oct 3, 2016

@Twonox implemented by sgdc3 in 88ce493, please verify

@Twonox
Copy link
Contributor Author

Twonox commented Oct 4, 2016

It's hard to explain in English, I will explain it in French, sorry Sgdc.

J'ai effectué des test et ça ne fonctionne pas très bien. J'ai rencontré plusieurs bug, et je ne peux pas tous les citer un-par-un. Tout ce que je peux te dire, c'est qu'au début de mes test ça avait l'air de fonctionner sauf pour antiBotSensibility qui déconnait lorsque je changeais sa valeur, et à l'instant où j'écris ce message l'antibot ne fonctionne plus du tout. J'ai beau faire /authme reload, /authme (switch/toggle)antibot on, ou même redémarrer le serveur, il ne s'active pas alors que j'ai enableAntiBot: true.

J'en ai profité pour réfléchir à un système de configuration un peu plus différent qui pourrai remplacer celui actuel si jamais vous n'arrivez pas à résoudre les soucis.
Je m'explique:

  • Vous remplacez enableAntiBot par antiBotMod qui prendra la valeur ON, OFF ou AUTO. Cette valeur sera modifiable depuis le serveur avec la commande/authme antibot ON/OFF/AUTO, en enlevant les autres commandes.
  • Comme les noms l'indiquent, ON sera pour garder constamment l'antibot actif, OFF pour qu'il soit constamment désactivé, et AUTO qui prendra en compte la configuration antiBotSensibility et antiBotDuration.

Edit: J'ai encore testé, cette fois ci l'antibot s'est bien activé mais lorsque j'ai changé la sensibilité ça a cessé de fonctionner.

@sgdc3
Copy link
Member

sgdc3 commented Oct 4, 2016

@Twonox Could you try again?

@ljacqu
Copy link
Member

ljacqu commented Oct 5, 2016

Quick translation of @Twonox's message is that Antibot doesn't seem to work well upon reloading.

It's important to know that Antibot takes 90 seconds before it actually starts; you can check this with /authme antibot (it shows DISABLED first and after that LISTENING). This should probably be configurable.

@sgdc3 fixed another bug with the commit just above, so if we could ask you to verify again @Twonox with that in mind. I create an issue for configuring the waiting period on startup in the meantime.

@Twonox
Copy link
Contributor Author

Twonox commented Oct 5, 2016

Okay I will do some test tomorrow.
The enabling delay is probably the cause of the negatives result, now I understand why the antibot doesn't work always.

@Twonox
Copy link
Contributor Author

Twonox commented Oct 6, 2016

It works perfectly, thanks ! :)

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

No branches or pull requests

5 participants
@Xephi @ljacqu @sgdc3 @Twonox and others