BlackDemonZyT / BotSentry

Official repository for Bot-Sentry issues, information, etc..
6 stars 0 forks source link

Kick : Lost connection: Received chat packet with missing or invalid signature #112

Closed Draghonir closed 8 months ago

Draghonir commented 1 year ago

Hello, I've just discovered a rather troublesome bug in BotSentry at version minecraft 1.20.2 and I'd like to explain the steps I took to confirm that it was indeed the culprit. When players connect and send a message in the chat, the message doesn't get sent to the server, and the players get kicked with the following message: "Lost connection: Received chat packet with missing or invalid signature." There were no other errors in the console. So, what I did was migrate all the plugins to a test server and removed them one by one to identify the responsible plugin. I found that it was BotSentry because when I reinstalled all the plugins except for BotSentry, everything worked fine. When I added BotSentry back, the same issue of the "kick lost connection: Received chat packet with missing or invalid signature" occurred. If necessary, I can provide video evidence to demonstrate this issue.

https://youtu.be/K60S-NZhLSI (No botsentry, no kick)

https://youtu.be/EwX7BoLM_VM (With botsentry, kick Lost connection: Received chat packet with missing or invalid signature)

The log capture is after the second video youtube

gf
Laurenshup commented 1 year ago

Hi, can you confirm you have detection.amazon.enabled set to true? What happens if you set it to false?

Draghonir commented 1 year ago

Hi,

yes my detection.amazon was set to true, after set to false bug was fix

what difference does it make to have this set to false Should I expect a drop in efficiency?

Laurenshup commented 1 year ago

Amazon checks whether players send some specific packets. While players haven't send it yet, any chat messages are blocked. Blocking chat messages is not allowed anymore on the proxy since the new encryption of chat messages in Minecraft.

The plugin will work the same, there is no drop in efficiency.

Draghonir commented 1 year ago

Alright, do you have plans in a future update to set it to 'false' by default to avoid this issue for players?"

Laurenshup commented 11 months ago

Hi @Draghonir, the setting for detection.amazon.enabled is already set to false now. We can change it remotely, which I have done one month ago. Therefore, when building a new config.yml, it should use the new setting.

Laurenshup commented 11 months ago

I will also include the information that the setting is not compatible with newer versions in the config.yml. It might also be an idea to include that this should mainly be used for the highest protection, but that it will most likely kick real players as well.

smscentras commented 11 months ago

All here have problem with detection.amazon when set to true I have lost all my players when your plugin kicked ALL online players.

Laurenshup commented 11 months ago

I have also heard people who have it working. But as said above. Most special clients do not send the packets correctly, which kicks them. But I will look into removing it.

smscentras commented 11 months ago

I think third party plugins are to blame here. Like ViaVersion or ProtocolLib.. Even bedrock clients were kicked out.

Laurenshup commented 8 months ago

Since amazon is disabled by default, and the message attached to it has changed in the newest version. Amazon has been completely removed from Spigot platforms, I believe this is done.