Skip to content

Commit 68288b7

Browse files
committed
Log warning on duplicate connection approved
1 parent d0e540a commit 68288b7

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

com.unity.netcode.gameobjects/Runtime/Messaging/Messages/ConnectionApprovedMessage.cs

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -247,14 +247,21 @@ public void Handle(ref NetworkContext context)
247247
{
248248
var networkManager = (NetworkManager)context.SystemOwner;
249249

250-
if (networkManager.CMBServiceConnection && networkManager.LocalClient.IsSessionOwner && networkManager.NetworkConfig.EnableSceneManagement && networkManager.LocalClientId != OwnerClientId)
250+
if (networkManager.CMBServiceConnection && networkManager.LocalClient.IsSessionOwner && networkManager.NetworkConfig.EnableSceneManagement)
251251
{
252-
if (NetworkLog.CurrentLogLevel <= LogLevel.Developer)
252+
if (networkManager.LocalClientId != OwnerClientId)
253253
{
254-
NetworkLog.LogInfo($"[Session Owner] Received connection approved for Client-{OwnerClientId}! Synchronizing...");
255-
}
254+
if (NetworkLog.CurrentLogLevel <= LogLevel.Developer)
255+
{
256+
NetworkLog.LogInfo($"[Session Owner] Received connection approved for Client-{OwnerClientId}! Synchronizing...");
257+
}
256258

257-
networkManager.SceneManager.SynchronizeNetworkObjects(OwnerClientId);
259+
networkManager.SceneManager.SynchronizeNetworkObjects(OwnerClientId);
260+
}
261+
else
262+
{
263+
NetworkLog.LogWarning($"[Client-{OwnerClientId}] Receiving duplicate connection approved. Client is already connected!");
264+
}
258265
return;
259266
}
260267

0 commit comments

Comments
 (0)