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

Server mode changes not tracked #498

Open
jacob1 opened this issue Dec 6, 2021 · 0 comments
Open

Server mode changes not tracked #498

jacob1 opened this issue Dec 6, 2021 · 0 comments

Comments

@jacob1
Copy link
Member

jacob1 commented Dec 6, 2021

Server mode changes are not tracked. This can lead to desyncs when net splits occur.

Noticed this today, during a netsplit:
[NET SPLIT]
Mon 12-06 [00:26:33] Kujou was mauled by wild animals and has died. New player count: 3
Mon 12-06 [00:26:33] iovoid was mauled by wild animals and has died. New player count: 2
[NET REJOIN]
Mon 12-06 [00:27:14] * osmium.libera.chat gives voice to Kujou iovoid
[GAME TIMEOUT]
Mon 12-06 [00:28:17] * lykos removes voice from pony jacob1
Mon 12-06 [00:28:17] PING! jacob1 pony
Mon 12-06 [00:28:18] The current game took too long to start and has been canceled. If you are still active, you can join again to start a new game.

Note that Kujou and iovoid were not devoiced, because it didn't think they were still voiced. Idealy they would have been devoiced during the net join though. !fsync did nothing to help the situation, I did confirm fsync itself works fine, it just wouldn't devoice those two ghost players.

The problem that this block returns before mode changes can be tracked:
https://github.com/lykoss/lykos/blob/master/src/hooks.py#L444
I am fairly certain comment is stale and no longer relevant. Ideally we should fire sync_modes after tracking the modes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants