Skip to content

Bug fix for 'PLAYER_DISCONNECTED' race condition#321

Open
steelskillet wants to merge 5 commits intoTeam-Silver-Sphere:masterfrom
AsgardEternal:upstream-playerdisconnectbug
Open

Bug fix for 'PLAYER_DISCONNECTED' race condition#321
steelskillet wants to merge 5 commits intoTeam-Silver-Sphere:masterfrom
AsgardEternal:upstream-playerdisconnectbug

Conversation

@steelskillet
Copy link
Contributor

@steelskillet steelskillet commented Nov 12, 2023

Fixes the bug described in #289.

Implements this by saving the list of oldPlayerInfo that updatePlayerList() creates and then the PLAYER_DISCONNECT event tries to find the player by steam id in the saved oldPlayerInfo list if it couldn't find it in this.players.

@steelskillet
Copy link
Contributor Author

Started testing on ASG servers.

@werewolfboy13
Copy link
Collaborator

Needs a rewrite for the new major squad js version

…sconnectbug

# Conflicts:
#	squad-server/index.js
@steelskillet
Copy link
Contributor Author

updated, needs further testing still.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants