feat: upgrade to lobby 1.1.0 [MTT-7092] [MTT-5665] [MTT-7101] [MTT-5708] (#860)

* updating to lobby 1.1.0

* locking lobby at creation to prevent other players from joining before it is ready

* removing explicit reference to wire in package manifest

* removing unnecessary periodic lobby updates

* removing old exception catch that is no longer necessary and adding new temporary catch for edge case exception

Author: LPLafontaineB

Assets/Scripts/ConnectionManagement/ConnectionMethod.cs

@@ -66,11 +66,11 @@ protected void SetConnectionPayload(string playerId, string playerName)
             m_ConnectionManager.NetworkManager.NetworkConfig.ConnectionData = payloadBytes;
         }
 
-        /// Using authentication, this makes sure your session is associated with your account and not your device. This means you could reconnect 
-        /// from a different device for example. A playerId is also a bit more permanent than player prefs. In a browser for example, 
+        /// Using authentication, this makes sure your session is associated with your account and not your device. This means you could reconnect
+        /// from a different device for example. A playerId is also a bit more permanent than player prefs. In a browser for example,
         /// player prefs can be cleared as easily as cookies.
-        /// The forked flow here is for debug purposes and to make UGS optional in Boss Room. This way you can study the sample without 
-        /// setting up a UGS account. It's recommended to investigate your own initialization and IsSigned flows to see if you need 
+        /// The forked flow here is for debug purposes and to make UGS optional in Boss Room. This way you can study the sample without
+        /// setting up a UGS account. It's recommended to investigate your own initialization and IsSigned flows to see if you need
         /// those checks on your own and react accordingly. We offer here the option for offline access for debug purposes, but in your own game you
         /// might want to show an error popup and ask your player to connect to the internet.
         protected string GetPlayerId()
@@ -156,7 +156,7 @@ public override async Task SetupClientConnectionAsync()
                 $"host: {joinedAllocation.HostConnectionData[0]} {joinedAllocation.HostConnectionData[1]}, " +
                 $"client: {joinedAllocation.AllocationId}");
 
-            await m_LobbyServiceFacade.UpdatePlayerRelayInfoAsync(joinedAllocation.AllocationId.ToString(), m_LocalLobby.RelayJoinCode);
+            await m_LobbyServiceFacade.UpdatePlayerDataAsync(joinedAllocation.AllocationId.ToString(), m_LocalLobby.RelayJoinCode);
 
             // Configure UTP with allocation
             var utp = (UnityTransport)m_ConnectionManager.NetworkManager.NetworkConfig.NetworkTransport;
@@ -197,13 +197,15 @@ public override async Task SetupHostConnectionAsync()
 
             m_LocalLobby.RelayJoinCode = joinCode;
 
-            //next line enable lobby and relay services integration
-            await m_LobbyServiceFacade.UpdateLobbyDataAsync(m_LocalLobby.GetDataForUnityServices());
-            await m_LobbyServiceFacade.UpdatePlayerRelayInfoAsync(hostAllocation.AllocationIdBytes.ToString(), joinCode);
+            // next line enables lobby and relay services integration
+            await m_LobbyServiceFacade.UpdateLobbyDataAndUnlockAsync();
+            await m_LobbyServiceFacade.UpdatePlayerDataAsync(hostAllocation.AllocationIdBytes.ToString(), joinCode);
 
             // Setup UTP with relay connection info
             var utp = (UnityTransport)m_ConnectionManager.NetworkManager.NetworkConfig.NetworkTransport;
             utp.SetRelayServerData(new RelayServerData(hostAllocation, k_DtlsConnType)); // This is with DTLS enabled for a secure connection
+
+            Debug.Log($"Created relay allocation with join code {m_LocalLobby.RelayJoinCode}");
         }
     }
 }

Assets/Scripts/ConnectionManagement/ConnectionState/HostingState.cs

@@ -129,7 +129,7 @@ public override void ApprovalCheck(NetworkManager.ConnectionApprovalRequest requ
             response.Reason = JsonUtility.ToJson(gameReturnStatus);
             if (m_LobbyServiceFacade.CurrentUnityLobby != null)
             {
-                m_LobbyServiceFacade.RemovePlayerFromLobbyAsync(connectionPayload.playerId, m_LobbyServiceFacade.CurrentUnityLobby.Id);
+                m_LobbyServiceFacade.RemovePlayerFromLobbyAsync(connectionPayload.playerId);
             }
         }
 

Assets/Scripts/ConnectionManagement/ConnectionState/StartingHostState.cs

@@ -68,7 +68,6 @@ async void StartHost()
             try
             {
                 await m_ConnectionMethod.SetupHostConnectionAsync();
-                Debug.Log($"Created relay allocation with join code {m_LocalLobby.RelayJoinCode}");
 
                 // NGO's StartHost launches everything
                 if (!m_ConnectionManager.NetworkManager.StartHost())

Assets/Scripts/UnityServices/Lobbies/JoinedLobbyContentHeartbeat.cs

@@ -42,6 +42,7 @@ public async Task<Lobby> CreateLobby(string requesterUasId, string lobbyName, in
             CreateLobbyOptions createOptions = new CreateLobbyOptions
             {
                 IsPrivate = isPrivate,
+                IsLocked = true, // locking the lobby at creation to prevent other players from joining before it is ready
                 Player = new Player(id: requesterUasId, data: hostUserData),
                 Data = lobbyData
             };