Skip to content

(fix): failed to reconnect after client crashed#1640

Merged
Kenshiin13 merged 24 commits intoesx-framework:devfrom
Tabby-Labs:dev
Jun 15, 2025
Merged

(fix): failed to reconnect after client crashed#1640
Kenshiin13 merged 24 commits intoesx-framework:devfrom
Tabby-Labs:dev

Conversation

@Tabby-Labs
Copy link
Copy Markdown
Contributor

@Tabby-Labs Tabby-Labs commented May 20, 2025

Description

This PR fixed failed to reconnecting after clients crashed, by validate if player id/source before connecting.


Motivation

The original validation for connecting logic just validate if ESX.GetPlayerFromIdentifier is in used, so when clients crashed, the playerDropped event isn't properly triggered and make the reconnecting failed.

Implementation Details

no new or modifies existing exposed functionality

Usage Example


PR Checklist

  • My commit messages and PR title follow the Conventional Commits standard.
  • My changes have been tested locally and function as expected.
  • My PR does not introduce any breaking changes.
  • I have provided a clear explanation of what my PR does, including the reasoning behind the changes and any relevant context.

@github-project-automation github-project-automation bot moved this to 🆕 To be Discussed in ESX Roadmap May 20, 2025
@github-project-automation github-project-automation bot moved this from 🆕 To be Discussed to In-Development in ESX Roadmap May 20, 2025
@Tabby-Labs Tabby-Labs requested a review from Kenshiin13 May 22, 2025 11:47
@Kenshiin13 Kenshiin13 self-assigned this Jun 11, 2025
Kenshiin13
Kenshiin13 previously approved these changes Jun 14, 2025
@github-project-automation github-project-automation bot moved this from In-Development to Ready for Next Update in ESX Roadmap Jun 15, 2025
@Kenshiin13 Kenshiin13 merged commit 7ce03c3 into esx-framework:dev Jun 15, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for Next Update

Development

Successfully merging this pull request may close these issues.

4 participants