Adding instructions when we get a service error on conflicting joins

Author: SamuelBellomo

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

@@ -21,6 +21,7 @@ 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;
@@ -67,7 +68,7 @@ void InjectDependenciesAndInitialize(AuthenticationServiceFacade authServiceFaca
             var arguments = System.Environment.GetCommandLineArgs();
             for (int i = 0; i < arguments.Length; i++)
             {
-                if (arguments[i] == "-AuthProfile")
+                if (arguments[i] == AuthProfileCommandLineArg)
                 {
                     var profileId = arguments[i + 1];
                     unityAuthenticationInitOptions.SetProfile(profileId);

Assets/BossRoom/Scripts/Client/UI/Lobby/LobbyUIMediator.cs

@@ -1,8 +1,10 @@
+using System;
 using TMPro;
 using Unity.Multiplayer.Samples.BossRoom.Client;
 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 Unity.Services.Lobbies;
 using Unity.Services.Lobbies.Models;
 using UnityEngine;
 
@@ -24,6 +26,7 @@ public class LobbyUIMediator : MonoBehaviour
         NameGenerationData m_NameGenerationData;
         GameNetPortal m_GameNetPortal;
         ClientGameNetPortal m_ClientNetPortal;
+        private IDisposable m_Subscriptions;
 
         [Inject]
         void InjectDependenciesAndInitialize(
@@ -33,7 +36,8 @@ void InjectDependenciesAndInitialize(
             LocalLobby localLobby,
             NameGenerationData nameGenerationData,
             GameNetPortal gameNetPortal,
-            ClientGameNetPortal clientGameNetPortal
+            ClientGameNetPortal clientGameNetPortal,
+            ISubscriber<UnityServiceErrorMessage> serviceError
         )
         {
             m_NameGenerationData = nameGenerationData;
@@ -46,6 +50,24 @@ ClientGameNetPortal clientGameNetPortal
             RegenerateName();
 
             m_ClientNetPortal.NetworkTimedOut += OnNetworkTimeout;
+            m_Subscriptions = serviceError.Subscribe(ServiceErrorHandler);
+        }
+
+        private void ServiceErrorHandler(UnityServiceErrorMessage error)
+        {
+            var errorMessage = error.Message;
+            if (error.AffectedService == UnityServiceErrorMessage.Service.Lobby)
+            {
+                if ((error.OriginalException as LobbyServiceException).Reason == LobbyExceptionReason.LobbyConflict)
+                {
+                    // 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");
+                }
+            }
+
+            PopupPanel.ShowPopupPanel("Service error", errorMessage);
         }
 
         void OnDestroy()
@@ -54,6 +76,8 @@ void OnDestroy()
             {
                 m_ClientNetPortal.NetworkTimedOut -= OnNetworkTimeout;
             }
+
+            m_Subscriptions.Dispose();
         }
 
         //Lobby and Relay calls done from UI

Assets/BossRoom/Scripts/Shared/Net/UnityServices/Auth/AuthenticationServiceFacade.cs

@@ -24,7 +24,7 @@ void OnServiceException(AuthenticationException e)
 
             var reason = $"{e.Message} ({e.InnerException?.Message})"; // Lobby error type, then HTTP error type.
 
-            m_UnityServiceErrorMessagePublisher.Publish(new UnityServiceErrorMessage("Authentication Error", reason));
+            m_UnityServiceErrorMessagePublisher.Publish(new UnityServiceErrorMessage("Authentication Error", reason, UnityServiceErrorMessage.Service.Authentication, e));
         }
 
         public void DoSignInAsync(Action onSigninComplete, Action onFailed, InitializationOptions initializationOptions)

Assets/BossRoom/Scripts/Shared/Net/UnityServices/Infrastructure/Messages/UnityServiceErrorMessage.cs

@@ -1,13 +1,26 @@
+using Unity.Services.Core;
+
 namespace Unity.Multiplayer.Samples.BossRoom.Shared.Net.UnityServices.Infrastructure
 {
     public struct UnityServiceErrorMessage
     {
+        public enum Service
+        {
+            Authentication,
+            Lobby,
+        }
+
         public string Title;
         public string Message;
-        public UnityServiceErrorMessage(string title, string message)
+        public Service AffectedService;
+        public RequestFailedException OriginalException;
+
+        public UnityServiceErrorMessage(string title, string message, Service service, RequestFailedException originalException = null)
         {
             Title = title;
             Message = message;
+            AffectedService = service;
+            OriginalException = originalException;
         }
     }
 }

Assets/BossRoom/Scripts/Shared/Net/UnityServices/Lobbies/LobbyAPIInterface.cs

@@ -66,7 +66,7 @@ void OnServiceException(LobbyServiceException e)
 
             var reason = $"{e.Message} ({e.InnerException?.Message})"; // Lobby error type, then HTTP error type.
 
-            m_UnityServiceErrorMessagePublisher.Publish(new UnityServiceErrorMessage("Lobby Error", reason));
+            m_UnityServiceErrorMessagePublisher.Publish(new UnityServiceErrorMessage("Lobby Error", reason, UnityServiceErrorMessage.Service.Lobby, e));
         }
 
         public void CreateLobbyAsync(string requesterUasId, string lobbyName, int maxPlayers, bool isPrivate, Dictionary<string, PlayerDataObject> hostUserData, Dictionary<string, DataObject> lobbyData, Action<Lobby> onComplete, Action onFailed)

Assets/BossRoom/Scripts/Shared/Net/UnityServices/Lobbies/LobbyServiceFacade.cs

@@ -124,7 +124,7 @@ void OnSuccess(Lobby lobby)
                             return;
                         }
                     }
-                    m_UnityServiceErrorMessagePub.Publish(new UnityServiceErrorMessage("Host left the lobby","Disconnecting."));
+                    m_UnityServiceErrorMessagePub.Publish(new UnityServiceErrorMessage("Host left the lobby","Disconnecting.", UnityServiceErrorMessage.Service.Lobby));
                     ForceLeaveLobbyAttempt();
                     m_ApplicationController.QuitGame();
                 }