PaperMC / Velocity

The modern, next-generation Minecraft server proxy.
https://papermc.io/software/velocity
GNU General Public License v3.0
1.68k stars 578 forks source link

Error with Packet size of protocols #1356

Open SimonMage opened 2 weeks ago

SimonMage commented 2 weeks ago

Hello, I have I problems on my server and using a plugin open source BetterHud I have made my custom hud on my backend server but Velocity kicks me from the proxy because the hud made with resourcepack is too much than Velociy accepts, as you can see on log. Please give me the possibility to increase that packet size or fix it. Obviously without Velocity as Proxy everything works fine.

2024-06-16-3.log

SimonMage commented 2 weeks ago

Log made with mcpaste.io

https://mcpaste.io/d735a60b8625d300

electronicboy commented 2 weeks ago

Looks like when they moved the packets over to the codec system they got rid of the packet specific component size limits

SimonMage commented 2 weeks ago

I'm not an expert but this means that can be fixed by Velocity right?

Looks like when they moved the packets over to the codec system they got rid of the packet specific component size limits

electronicboy commented 2 weeks ago

Yes, we just need to increase the limits for versions after that change; I just can't touch much because I'm recovering from surgery

SimonMage commented 2 weeks ago

Thank you so much and I hope you to recover faster, good health!

Yes, we just need to increase the limits for versions after that change; I just can't touch much because I'm recovering from surgery

SimonMage commented 2 weeks ago

Hey 👋🏻 any news about it? It was just needed to increase the limits right?

SimonMage commented 2 weeks ago

Screenshot_1

I noticed that in some cases the size is variable, in this case is over 90.000.

Please increase to a number very big or make a config in velocity.toml to change the default value for future problems

Yes, we just need to increase the limits for versions after that change; I just can't touch much because I'm recovering from surgery

electronicboy commented 2 weeks ago

We are not going to expose a config option. When this is fixed, the packet limit will reflect the protocol network limit as determined by the vanilla spec.

SimonMage commented 2 weeks ago

We are not going to expose a config option. When this is fixed, the packet limit will reflect the protocol network limit as determined by the vanilla spec.

Oh really? And how much is the packet limit for Minecraft 1.20.2 ?

SimonMage00 commented 22 hours ago

We are not going to expose a config option. When this is fixed, the packet limit will reflect the protocol network limit as determined by the vanilla spec.

Any news about htis update?