Adding back GUID with IP connections

When not using UGS, player id is now defined as GUID + profile string

Author: LPLafontaineB

Assets/BossRoom/Scripts/Client/Game/State/ClientMainMenuState.cs

@@ -6,9 +6,6 @@
 using Unity.Services.Core;
 using UnityEngine;
 
-#if UNITY_EDITOR
-using ParrelSync;
-#endif
 
 namespace Unity.Multiplayer.Samples.BossRoom.Client
 {
@@ -21,7 +18,6 @@ namespace Unity.Multiplayer.Samples.BossRoom.Client
     /// </remarks>
     public class ClientMainMenuState : GameStateBehaviour
     {
-        public const string AuthProfileCommandLineArg = "-AuthProfile";
         public override GameState ActiveState { get { return GameState.MainMenu; } }
 
         [SerializeField] GameObject[] m_GameObjectsThatWillBeInjectedAutomatically;
@@ -49,33 +45,11 @@ void InjectDependenciesAndInitialize(AuthenticationServiceFacade authServiceFaca
             m_Scope.BindInstanceAsSingle(m_LobbyUIMediator);
 
             var unityAuthenticationInitOptions = new InitializationOptions();
-
-#if UNITY_EDITOR
-            //The code below makes it possible for the clone instance to log in as a different user profile in Authentication service.
-            //This allows us to test services integration locally by utilising Parrelsync.
-            if (ClonesManager.IsClone())
-            {
-                Debug.Log("This is a clone project.");
-                var customArguments = ClonesManager.GetArgument().Split(',');
-
-                //second argument is our custom ID, but if it's not set we would just use some default.
-
-                var hardcodedProfileID = customArguments.Length > 1 ? customArguments[1] : "defaultCloneID";
-
-                unityAuthenticationInitOptions.SetProfile(hardcodedProfileID);
-            }
-#else
-            var arguments = System.Environment.GetCommandLineArgs();
-            for (int i = 0; i < arguments.Length; i++)
+            var profile = ProfileManager.Profile;
+            if (profile.Length > 0)
             {
-                if (arguments[i] == AuthProfileCommandLineArg)
-                {
-                    var profileId = arguments[i + 1];
-                    unityAuthenticationInitOptions.SetProfile(profileId);
-                    break;
-                }
+                unityAuthenticationInitOptions.SetProfile(profile);
             }
-#endif
 
             authServiceFacade.DoSignInAsync(OnAuthSignIn,  OnSignInFailed, unityAuthenticationInitOptions);
 

Assets/BossRoom/Scripts/Client/UI/UnityServicesUIHandler.cs

@@ -32,7 +32,7 @@ void ServiceErrorHandler(UnityServiceErrorMessage error)
                     // LobbyConflict can have multiple causes. Let's add other solutions here if there's other situations that arise for this.
                     errorMessage += "\nSee logs for possible causes and solution.";
                     Debug.LogError($"Got service error {error.Message} with LobbyConflict. Possible conflict cause: Trying to play with two builds on the " +
-                                   $"same machine. Please use command line arg '{ClientMainMenuState.AuthProfileCommandLineArg} someName' to set a different auth profile.\n");
+                                   $"same machine. Please use command line arg '{ProfileManager.AuthProfileCommandLineArg} someName' to set a different auth profile.\n");
                 }
             }
 
@@ -44,4 +44,4 @@ void OnDestroy()
             m_Subscriptions.Dispose();
         }
     }
-}
+}

Assets/BossRoom/Scripts/Client/com.unity.multiplayer.samples.bossroom.client.asmdef

@@ -12,8 +12,7 @@
         "Photon Realtime Transport for Netcode for GameObjects",
         "Unity.Services.Relay",
         "Unity.Multiplayer.Samples.Utilities",
-        "Unity.Services.Lobbies",
-        "ParrelSync"
+        "Unity.Services.Lobbies"
     ],
     "includePlatforms": [],
     "excludePlatforms": [],
@@ -24,4 +23,4 @@
     "defineConstraints": [],
     "versionDefines": [],
     "noEngineReferences": false
-}
+}

Assets/BossRoom/Scripts/Shared/Net/ConnectionManagement/ClientGameNetPortal.cs

@@ -1,6 +1,5 @@
 using System;
 using Unity.Multiplayer.Samples.BossRoom.Shared.Infrastructure;
-using Unity.Multiplayer.Samples.BossRoom.Shared.Net.UnityServices.Infrastructure;
 using Unity.Multiplayer.Samples.BossRoom.Shared.Net.UnityServices.Lobbies;
 using UnityEngine;
 using UnityEngine.SceneManagement;
@@ -19,6 +18,9 @@ public class ClientGameNetPortal : MonoBehaviour
         public static ClientGameNetPortal Instance;
         private GameNetPortal m_Portal;
 
+        string m_JoinCode;
+        OnlineMode m_OnlineMode;
+
         /// <summary>
         /// If a disconnect occurred this will be populated with any contextual information that was available to explain why.
         /// </summary>
@@ -156,11 +158,11 @@ private void OnDisconnectOrTimeout(ulong clientID)
         /// This method must be static because, when it is invoked, the client still doesn't know it's a client yet, and in particular, GameNetPortal hasn't
         /// yet initialized its client and server GNP-Logic objects yet (which it does in OnNetworkSpawn, based on the role that the current player is performing).
         /// </remarks>
-        /// <param name="portal"> </param>
         /// <param name="ipaddress">the IP address of the host to connect to. (currently IPV4 only)</param>
         /// <param name="port">The port of the host to connect to. </param>
         public void StartClient(string ipaddress, int port)
         {
+            m_OnlineMode = OnlineMode.IpHost;
             var chosenTransport = NetworkManager.Singleton.gameObject.GetComponent<TransportPicker>().IpHostTransport;
             NetworkManager.Singleton.NetworkConfig.NetworkTransport = chosenTransport;
 
@@ -178,39 +180,52 @@ public void StartClient(string ipaddress, int port)
                     throw new ArgumentOutOfRangeException(nameof(chosenTransport));
             }
 
-            ConnectClient();
+            ConnectClient(null);
         }
 
-        public async void StartClientUnityRelayModeAsync(string joinCode, Action<string> onFailure)
+        public void StartClientUnityRelayModeAsync(string joinCode, Action<string> onFailure)
         {
-            var utp = (UnityTransport)NetworkManager.Singleton.gameObject.GetComponent<TransportPicker>().UnityRelayTransport;
+            m_OnlineMode = OnlineMode.UnityRelay;
+            m_JoinCode = joinCode;
+            var utp = NetworkManager.Singleton.gameObject.GetComponent<TransportPicker>().UnityRelayTransport;
             NetworkManager.Singleton.NetworkConfig.NetworkTransport = utp;
 
             Debug.Log($"Setting Unity Relay client with join code {joinCode}");
 
-            try
+            ConnectClient(onFailure);
+        }
+
+        async void ConnectClient(Action<string> onFailure)
+        {
+            string playerId;
+            if (m_OnlineMode == OnlineMode.UnityRelay)
             {
-                var clientRelayUtilityTask =  UnityRelayUtilities.JoinRelayServerFromJoinCode(joinCode);
-                await clientRelayUtilityTask;
-                var (ipv4Address, port, allocationIdBytes, connectionData, hostConnectionData, key) = clientRelayUtilityTask.Result;
+                try
+                {
+                    var clientRelayUtilityTask =  UnityRelayUtilities.JoinRelayServerFromJoinCode(m_JoinCode);
+                    await clientRelayUtilityTask;
+                    var (ipv4Address, port, allocationIdBytes, connectionData, hostConnectionData, key) = clientRelayUtilityTask.Result;
 
-                m_LobbyServiceFacade.UpdatePlayerRelayInfoAsync(allocationIdBytes.ToString(), joinCode, null, null);
-                utp.SetClientRelayData(ipv4Address, port, allocationIdBytes, key, connectionData, hostConnectionData, isSecure: true);
+                    m_LobbyServiceFacade.UpdatePlayerRelayInfoAsync(allocationIdBytes.ToString(), m_JoinCode, null, null);
+                    var utp = (UnityTransport) NetworkManager.Singleton.NetworkConfig.NetworkTransport;
+                    utp.SetClientRelayData(ipv4Address, port, allocationIdBytes, key, connectionData, hostConnectionData, isSecure: true);
+                }
+                catch (Exception e)
+                {
+                    onFailure?.Invoke(e.Message);
+                    return;//not re-throwing, but still not allowing to connect
+                }
+
+                playerId = AuthenticationService.Instance.PlayerId;
             }
-            catch (Exception e)
+            else
             {
-                onFailure?.Invoke(e.Message);
-                return;//not re-throwing, but still not allowing to connect
+                playerId = ClientPrefs.GetGuid() + ProfileManager.Profile;
             }
 
-            ConnectClient();
-        }
-
-        private void ConnectClient()
-        {
             var payload = JsonUtility.ToJson(new ConnectionPayload()
             {
-                playerId = AuthenticationService.Instance.PlayerId,
+                playerId = playerId,
                 clientScene = SceneManager.GetActiveScene().buildIndex,
                 playerName = m_Portal.PlayerName
             });

Assets/BossRoom/Scripts/Shared/ProfileManager.cs

@@ -0,0 +1,59 @@
+using UnityEngine;
+
+#if UNITY_EDITOR
+using ParrelSync;
+#endif
+
+public static class ProfileManager
+{
+    public const string AuthProfileCommandLineArg = "-AuthProfile";
+
+    static bool s_IsProfileSet = false;
+
+    static string s_Profile = "";
+
+    public static string Profile
+    {
+        get
+        {
+            if (!s_IsProfileSet)
+            {
+                s_Profile = GetProfile();
+                s_IsProfileSet = true;
+            }
+
+            return s_Profile;
+        }
+        private set => s_Profile = value;
+    }
+
+    static string GetProfile()
+    {
+#if UNITY_EDITOR
+            //The code below makes it possible for the clone instance to log in as a different user profile in Authentication service.
+            //This allows us to test services integration locally by utilising Parrelsync.
+            if (ClonesManager.IsClone())
+            {
+                Debug.Log("This is a clone project.");
+                var customArguments = ClonesManager.GetArgument().Split(',');
+
+                //second argument is our custom ID, but if it's not set we would just use some default.
+
+                var hardcodedProfileID = customArguments.Length > 1 ? customArguments[1] : "defaultCloneID";
+
+                return hardcodedProfileID;
+            }
+#else
+        var arguments = System.Environment.GetCommandLineArgs();
+        for (int i = 0; i < arguments.Length; i++)
+        {
+            if (arguments[i] == AuthProfileCommandLineArg)
+            {
+                var profileId = arguments[i + 1];
+                return profileId;
+            }
+        }
+#endif
+        return "";
+    }
+}

Assets/BossRoom/Scripts/Shared/ProfileManager.cs.meta

@@ -12,7 +12,8 @@
         "Unity.Collections",
         "Photon Realtime Transport for Netcode for GameObjects",
         "Unity.Multiplayer.Samples.Utilities",
-        "Unity.Networking.Transport"
+        "Unity.Networking.Transport",
+        "ParrelSync"
     ],
     "includePlatforms": [],
     "excludePlatforms": [],
@@ -23,4 +24,4 @@
     "defineConstraints": [],
     "versionDefines": [],
     "noEngineReferences": false
-}
+}