EpicOnlineTransport icon indicating copy to clipboard operation
EpicOnlineTransport copied to clipboard

EosTransport should call OnClientDisconnected when client disconnects

Open Frechfuchs opened this issue 2 years ago • 0 comments
trafficstars

Talking about these lines: https://github.com/FakeByte/EpicOnlineTransport/blob/master/Mirror/Runtime/Transport/EpicOnlineTransport/EosTransport.cs#L160-L163

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?

Frechfuchs avatar Jul 11 '23 18:07 Frechfuchs