ClientId is 0 (or default) in the OnClientDisconnectedCallback when client is refused immediately

[LeFlown]

Description

ClientId is 0 (or default) in the OnClientDisconnectedCallback if server refuse the connection immediately

Reproduce Steps

• Start a server with a simple approval callback, no client connected.
• Connect a client.
• In the ConnectionApprovalCallback (server-side), the ClientId is 1 (request.ClientNetworkId)
• Refused the client immediately (Version mismatch for example)
• In the OnClientDisconnectCallback, observe the clientId parameter value, 0 instead of 1.

Actual Outcome

Wrong client Id in the OnClientDisconnectedCallback parameter

Expected Outcome

Good client Id in the OnClientDisconnectedCallback parameter

Screenshots

Showing that the TransportIdCleanUp is called twice, returning the wrong value the second time:

Environment

• OS: Windows 10
• Unity Version: 2022.2.3
• Netcode Version: 1.4

[NoelStephensUnity]

@LeFlown
The develop branch is a little different now, but I was still able to replicate it and I have already created a fix for this issue.
Will be making a PR for this shortly.

[LeFlown]

Thank you!