Skip to content
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

Players desync and timeout #578

Closed
SKITTLE6969 opened this issue Sep 21, 2024 · 4 comments
Closed

Players desync and timeout #578

SKITTLE6969 opened this issue Sep 21, 2024 · 4 comments
Labels
bug Something isn't working need repro This bug report needs confirmation

Comments

@SKITTLE6969
Copy link
Contributor

SKITTLE6969 commented Sep 21, 2024

Summary

When player count hits 64+ this function starts to spam and players starts to desync and timeout

local function playerStateBagCheck(bagName, meta, value)

Reproduction

Update to latest and test with at least 70 people

Expected behavior

No timeout whatsoever

Actual behavior

All the players gets timed out, refer to the profiler data attached below 👇

profileData_16.json

Additional context

No response

Current Version

Latest

Custom Resources

N/a

@SKITTLE6969 SKITTLE6969 added bug Something isn't working need repro This bug report needs confirmation labels Sep 21, 2024
@qbox-duck qbox-duck bot added this to Issues Sep 21, 2024
@github-project-automation github-project-automation bot moved this to Todo in Issues Sep 21, 2024
@SynoFPS
Copy link

SynoFPS commented Sep 22, 2024

Same issue somewhere along the same lines but with QBCore:Player:SetPlayerData and qbx_core:client:onSetMetadata being spammed in network event log and timing out users/crashing server
image (4)

@SKITTLE6969
Copy link
Contributor Author

Same issue somewhere along the same lines but with QBCore:Player:SetPlayerData and qbx_core:client:onSetMetadata being spammed in network event log and timing out users/crashing server image (4)

revert this pr and try 6007cb9

@SynoFPS
Copy link

SynoFPS commented Sep 22, 2024

this did the trick net event log cleared off those events, thank you very much for the suggestion!

@SKITTLE6969
Copy link
Contributor Author

thanks @solareon <3
closing in favour of 07528de

@github-project-automation github-project-automation bot moved this from Todo to Done in Issues Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need repro This bug report needs confirmation
Projects
Status: Done
Development

No branches or pull requests

2 participants