feat: adding list for AuthenticatedPlayers in NetworkServer

Adding extra list for authenticated players so that can be looped over instead of all connections

Assets/Mirage/Runtime/NetworkServer.cs

@@ -132,11 +132,19 @@ public class NetworkServer : MonoBehaviour
         public bool IsHost => LocalClient != null && LocalClient.Active;
 
         /// <summary>
-        /// A list of connections on the server.
+        /// All players on server (including unauthenticated players)
         /// </summary>
+        public IReadOnlyCollection<INetworkPlayer> AllPlayers => _connections.Values;
+        [Obsolete("Use AllPlayers or AuthenticatedPlayers instead")]
         public IReadOnlyCollection<INetworkPlayer> Players => _connections.Values;
 
+        /// <summary>
+        /// List of players that have Authenticated with server
+        /// </summary>
+        public IReadOnlyList<INetworkPlayer> AuthenticatedPlayers => _authenticatedPlayers;
+
         private readonly Dictionary<IConnection, INetworkPlayer> _connections = new Dictionary<IConnection, INetworkPlayer>();
+        private readonly List<INetworkPlayer> _authenticatedPlayers = new List<INetworkPlayer>();
 
         /// <summary>
         /// <para>Checks if the server has been started.</para>
@@ -186,6 +194,7 @@ public void Stop()
 
             // just clear list, connections will be disconnected when peer is closed
             _connections.Clear();
+            _authenticatedPlayers.Clear();
             LocalClient = null;
             LocalPlayer = null;
 
@@ -229,7 +238,7 @@ public void StartServer(NetworkClient localClient = null)
             Application.quitting += Stop;
             if (logger.LogEnabled()) logger.Log($"NetworkServer created, Mirage version: {Version.Current}");
 
-            logger.Assert(Players.Count == 0, "Player should have been reset since previous session");
+            logger.Assert(_authenticatedPlayers.Count == 0, "Player should have been reset since previous session");
             logger.Assert(_connections.Count == 0, "Connections should have been reset since previous session");
 
             World = new NetworkWorld();
@@ -413,6 +422,7 @@ private void AuthenticationSuccess(INetworkPlayer player, AuthenticationResult r
             player.Send(new AuthSuccessMessage { AuthenticatorName = result.Authenticator?.AuthenticatorName });
 
             // add connection
+            _authenticatedPlayers.Add(player);
             Authenticated?.Invoke(player);
         }
 
@@ -438,6 +448,7 @@ private void Peer_OnDisconnected(IConnection conn, DisconnectReason reason)
         private void RemoveConnection(INetworkPlayer player)
         {
             _connections.Remove(player.Connection);
+            _authenticatedPlayers.Remove(player);
         }
 
         /// <summary>