fix

This resolves an issue where a very small memory leak could occur within the ConnectionApprovedMessage.

Author: NoelStephensUnity

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

@@ -262,6 +262,7 @@ public void Handle(ref NetworkContext context)
                 {
                     NetworkLog.LogWarning($"[Client-{OwnerClientId}] Receiving duplicate connection approved. Client is already connected!");
                 }
+                ConnectedClientIds.Dispose();
                 return;
             }
 
@@ -304,6 +305,9 @@ public void Handle(ref NetworkContext context)
                 }
             }
 
+            // Dispose after it has been used.
+            ConnectedClientIds.Dispose();
+
             // Only if scene management is disabled do we handle NetworkObject synchronization at this point
             if (!networkManager.NetworkConfig.EnableSceneManagement)
             {
@@ -387,7 +391,6 @@ public void Handle(ref NetworkContext context)
                     }
                 }
             }
-            ConnectedClientIds.Dispose();
         }
     }
 }