ClientDisconnect should call OnClientDisconnected, which will call NetworkClient.OnTransportDisconnected, which will removes transport handlers, which clients need to remove anything registered in AddTransportHandlers.
An easy fix would be these lines:
public override void ClientDisconnect()
{
if (!ClientActive()) return;
Shutdown();
OnClientDisconnected();
}
Is there a reason not to call OnClientDisconnected here?
Talking about these lines: https://github.com/FakeByte/EpicOnlineTransport/blob/master/Mirror/Runtime/Transport/EpicOnlineTransport/EosTransport.cs#L160-L163
ClientDisconnectshould callOnClientDisconnected, which will callNetworkClient.OnTransportDisconnected, which will removes transport handlers, which clients need to remove anything registered inAddTransportHandlers.An easy fix would be these lines:
Is there a reason not to call
OnClientDisconnectedhere?