Citizens2: Kicked from server for "Invalid Player Move Packet Received" inconsistently when using any kind of teleport. Issue stops when essentials or citizens is removed.
/version output
his server is running Paper version git-Paper-81 (MC: 1.19) (Implementing API version 1.19-R0.1-SNAPSHOT) (Git: 86f87ba)
[25.07 22:30:30] [Server] You are running the latest version
[25.07 22:30:30] [Server] Previous versiongit-Paper-78 (MC: 1.19)
/version Citizens output
Citizens version 2.0.30-SNAPSHOT (build 2633)
[25.07 22:31:19] [Server] [INFO] Website: http://www.citizensnpcs.co
[25.07 22:31:19] [Server] [INFO] Author: fullwall
Log file
N/A
config.yml
https://paste.denizenscript.com/View/99387
saves.yml
https://paste.denizenscript.com/View/99388
Profiler report
N/A
Description
No error in the console. Players are inconsistently kicked whenever they use any kind of teleport after the 1.19 update. Teleports include /tp, /tpa, /warp, /spawn, /rtp, and even nether portals. The kick they get is for: “Invalid player move packet received”.
There is no error logs in console.
I found that when removing either essentials or citizens the problem goes away.
I found another server owner who also has had the same bug as me. I tested every other plugin we shared between us, but no others fixed the issue when they were removed. The other server owner kick events happened less on their server than mine. We both had similar sizes of RAM and no issue with tps for our servers. They were a smaller server though.
We imported the other server owners’ files on a test server, but couldn’t confirm if the bug was present or not because it was much more of a rare occurrence for him. I tried testing my essentials and citizen server133597 - 25 Jul 2022 17_30_0.zip s
About this issue
- Original URL
- State: closed
- Created 2 years ago
- Comments: 28 (11 by maintainers)
We are also receiving this issue. Discussed a bit with electroniccat in #paper-help if you want to read it. With this patch applied, this is our log: https://paste.denizenscript.com/View/99432
What seems to be happening is Citizens is modifying the
awaitingPositionFromClientvariable, setting it to null.The player kick is only happening after this commit in paper: https://github.com/PaperMC/Paper/commit/5deafd1969b3c521af9cabb46822ecb01e6ce498
If it relates to Denizen it would make more sense than Citizens - Denizen has a network interception tool (that is enabled when you do
mirrorskinor similar) which could potentially muck with that data - I’m confused about the reports that don’t include Denizen if that’s the case thoughEDIT: I can replicate this, will investigate further
EDIT2: More reports are coming in on Paper side that don’t have Denizen at all. I’m suspecting this is a Paper bug that can be exacerbated by multiple different causes, and Denizen is one of those exacerbation cause rather than the source itself. The bug replicates consistently/reliably on Paper, does not replicate on Spigot at all. Nonetheless I will continue to try to investigate - will likely require hunting through Paper source code, which will be painful but necessary to track this down.