Double perks on disconnect & rejoin

Hi there,

Love the plugin

One of our players has had an interesting thing happen - he was disconnected from the server while AFK and relogged later - during that time he was killed for being in a safezone, on return it seems that some or all of his skills were being applied twice

See the attached image - they had 3 points in the Quality Crafter skill and was able to get 6 slots on their armour!

Reloading the plugin seems to have resolved it but the player also quit the game and relogged again fresh at the same time so that may also have done the trick

Thanks!

Heya,

Well I can reproduce it on demand, but it's very very specific circumstances

-Go to safezone
-Teleport away from safezone, but don't wake up (using TeleportGUI plugin for teleportation/homes)
-After some time as a sleeper, disconnected with error, then killed for being asleep in safezone (this is a known issue it seems with TeleportGUI which I will report to that dev as well) I will add disconnection error below - the disconnect and the death appear to occur at the same time, hard to say as there is no output on console on death, but if I immediately rejoin on disconnect I am dead so assume this happens simultaneously...
-Reconnect
-Have double skills (see attached before and after pic)
-Disconnect & reconnect after that fixes the problem

If you disconnect yourself or time out to disconnect, the issue does not occur

Seems something to do with the hook/method or whatever for disconnection not catching when the client DCs in this fashion?

NullReferenceException: Object reference not set to an instance of an object 02/01 09:58:57 | *IP*:32848/*steamid*/*steamname* disconnecting: Exception (cs:141825): NullReferenceException: Object reference not set to an instance of an object. BasePlayer.%b916d9d124b8e9856bf396d6a3b08881d5c984ca (%4f36be8fe088287d2718bb52e458fbf5a3f30083+%8700274fac70c49688d318b85d9413a6d672a04c& %80e50159f6c6e37a9152322fcf34310da97a5859) (at <00000000000000000000000000000000>:0) BasePlayer.%b916d9d124b8e9856bf396d6a3b08881d5c984ca () (at <00000000000000000000000000000000>:0) BasePlayer.%b82a23817908f9f94ddcc55d348847d9e100b8c5 (%04a567ddb4324f873a78cf0daadc74937bc2d517 %aa2004f11a7681caa25138ab8f52b90b5e4b42cb, System.Single %17f124108775211a04d0647435624f3f35e4c113) (at <00000000000000000000000000000000>:0) BasePlayer.ClientUpdateLocalPlayer (System.Single %fcfa3dc15fda61cce1013c440923752458e1bcd2) (at <00000000000000000000000000000000>:0) BasePlayer.%a187206330b0329d6334947dae1b66ad71ae584a (System.Single %fcfa3dc15fda61cce1013c440923752458e1bcd2) (at <00000000000000000000000000000000>:0) Client.Update () (at <00000000000000000000000000000000>:0)

Appreciate we can only replicate it by abusing behaviour/bugs of another plugin but possible there are other ways to force this kind of disconnect to cheese ST?

Cheers