small bug fixes

* shutting down host now changes state to offline
* fixed return value of JoinRelayServerAsync

(cherry picked from commit 593da8a)

Author: LPLafontaineB

Assets/Scripts/Gameplay/ConnectionManagement/ConnectionState/HostingConnectionState.cs

@@ -147,6 +147,7 @@ IEnumerator WaitToShutdown()
             yield return null;
             m_ConnectionManager.NetworkManager.Shutdown();
             SessionManager<SessionPlayerData>.Instance.OnServerEnded();
+            m_ConnectionManager.ChangeState(ConnectionStateType.Offline);
         }
     }
 }

Assets/Scripts/Gameplay/ConnectionManagement/ConnectionState/OfflineConnectionState.cs

@@ -45,10 +45,11 @@ public override void StartClientIP(string playerName, string ipaddress, int port
 
         public override async Task StartClientLobbyAsync(string playerName, Action<string> onFailure)
         {
-            if (await JoinRelayServerAsync(onFailure)) return;//not re-throwing, but still not allowing to connect
-
-            ConnectClient(GetPlayerId(), playerName);
-            m_ConnectionManager.ChangeState(ConnectionStateType.Connecting);
+            if (await JoinRelayServerAsync(onFailure))
+            {
+                ConnectClient(GetPlayerId(), playerName);
+                m_ConnectionManager.ChangeState(ConnectionStateType.Connecting);
+            }
         }
 
         protected async Task<bool> JoinRelayServerAsync(Action<string> onFailure)
@@ -67,10 +68,10 @@ protected async Task<bool> JoinRelayServerAsync(Action<string> onFailure)
             catch (Exception e)
             {
                 onFailure?.Invoke(e.Message);
-                return true;
+                return false;
             }
 
-            return false;
+            return true;
         }
 
         public override bool StartHostIP(string playerName, string ipaddress, int port)