-
Notifications
You must be signed in to change notification settings - Fork 0
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
FastLogin incompatible on Bungee-like systems #21
Comments
I am in need of more information about this issue. BotSentry is definitely cancelling the PreLoginEvent of BungeeCord. FastLogin should not do any checks. |
If you need any information like log let me know, I will try to help as much as possible |
Any logs are welcome. Also can you say which version of BotSentry you are using? |
Logs: https://mclo.gs/4ugqi3b |
@SrBedrock Can you give me the line number in which you think the attack started? |
It was already occurring, the log is after a restart to disable a plugin that was causing problems, they will probably solve the problem soon so I hope. If you want the previous logs let me know, but the server was inaccessible due to the amount of requests with the mojang and database by the anti-vpn system. |
So. BotSentry did not have any problems with FastLogin? Because it does not seem so. |
@Laurenshup Is your plugin performing the checks asynchronous. Then setting the cancelled state could occur after the event already fired for other plugins. |
Nope, the plugin checks both synchronous/asynchronous. But inside of the PreLoginEvent, it always does the basic checks (checking if IP is in blacklist, etc.) synchronous therefore it also sets the cancelled state synchronous. |
Some updates about this issue. The API in the upcoming versions, will use CompletableFuture, allowing for thread-blocking on PreLoginEvent by other plugins. This will help them to check for the BotSentry status first. Further it seems like FastLogin uses the normal Priority on Bungee. BotSentry uses the lowest, meaning, BotSentry will come first. |
A customer, has experienced this bug. And the best way to solve this problem is by directly implementing into the FastLoginBungee API. |
The issue has been reported at:
games647/FastLogin#516
The text was updated successfully, but these errors were encountered: