Skip to content

fix: ignore clients not in game in OnClientDisconnect - #31

Open
cmer81 wants to merge 1 commit into
masterfrom
fix/disconnect-not-in-game
Open

cmer81 wants to merge 1 commit into
masterfrom
fix/disconnect-not-in-game

Conversation

@cmer81

@cmer81 cmer81 commented Oct 6, 2026

Copy link
Copy Markdown
Member

OnClientDisconnect fires for players that disconnect before being in game (timeout, rejection, map change). Calling VIP_IsClientVIP on them makes VIP-Core's CheckValidClient throw Client N is not connected, producing a lot of error logs.

Add an IsClientInGame guard (such a player can't have received the test VIP anyway) and bump version to 2.1.1.

Compiled with spcomp 1.13.0.7427, no errors or warnings.

VIP_IsClientVIP throws a native error ('Client N is not connected') when
the client never reached in-game state, flooding the logs.
@cmer81
cmer81 requested a review from Rushaway October 6, 2026 15:12
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.

1 participant