Tiny fixes with hostnames and web/api urls

github-for-unity · Oct 23, 2018 · 94d7fa1 · 94d7fa1
1 parent c7c9a15
commit 94d7fa1
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/src/GitHub.Api/Application/ApiClient.cs b/src/GitHub.Api/Application/ApiClient.cs
@@ -262,7 +262,7 @@ public void LoginWithToken(string token, Action<bool> result)
             Guard.ArgumentNotNull(result, "result");
 
             new FuncTask<bool>(taskManager.Token,
-                    () => loginManager.LoginWithToken(HostAddress.WebUri.Host, token))
+                    () => loginManager.LoginWithToken(UriString.ToUriString(HostAddress.WebUri), token))
                 .FinallyInUI((success, ex, res) =>
                 {
                     if (!success)
@@ -279,7 +279,7 @@ public void LoginWithToken(string token, Action<bool> result)
 
         public void CreateOAuthToken(string code, Action<bool, string> result)
         {
-            var command = "token -h " + HostAddress.ApiUri.Host;
+            var command = "token -h " + HostAddress.WebUri.Host;
             var octorunTask = new OctorunTask(taskManager.Token, environment, command, code)
                 .Configure(processManager);
 
@@ -388,7 +388,7 @@ private Connection Connection
             {
                 if (connection == null)
                 {
-                    connection = keychain.Connections.FirstOrDefault(x => x.Host == (UriString)HostAddress.WebUri.Host);
+                    connection = keychain.Connections.FirstOrDefault(x => x.Host.ToUriString().Host == HostAddress.WebUri.Host);
                 }
 
                 return connection;

diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/GitHubAuthenticationView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/GitHubAuthenticationView.cs
@@ -216,7 +216,7 @@ private void OnOAuthCallback(string state, string code)
             if (state.Equals(oAuthState))
             {
                 isBusy = true;
-                authenticationService.LoginWithOAuthCode(code, DoOAuthCodeResult);
+                authenticationService.LoginWithOAuthCode(code, (b, s) => TaskManager.RunInUI(() => DoOAuthCodeResult(b, s)));
             }
         }
 
@@ -292,7 +292,7 @@ private AuthenticationService AuthenticationService
             {
                 if (authenticationService == null)
                 {
-                    AuthenticationService = new AuthenticationService(HostAddress.GitHubDotComHostAddress.WebUri.Host, Platform.Keychain, Manager.ProcessManager, Manager.TaskManager, Environment);
+                    AuthenticationService = new AuthenticationService(UriString.ToUriString(HostAddress.GitHubDotComHostAddress.WebUri), Platform.Keychain, Manager.ProcessManager, Manager.TaskManager, Environment);
                 }
                 return authenticationService;
             }

diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/GitHubEnterpriseAuthenticationView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/GitHubEnterpriseAuthenticationView.cs
@@ -334,7 +334,7 @@ private void OnOAuthCallback(string state, string code)
                 if (state.Equals(oAuthState))
                 {
                     isBusy = true;
-                    authenticationService.LoginWithOAuthCode(code, DoOAuthCodeResult);
+                    authenticationService.LoginWithOAuthCode(code, (b, s) => TaskManager.RunInUI(() => DoOAuthCodeResult(b, s)));
                 }
             });
         }

diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/PublishView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/PublishView.cs
@@ -59,7 +59,7 @@ private void MaybeUpdateData()
             {
                 connectionsNeedLoading = false;
                 connections = Platform.Keychain.Connections.OrderByDescending(HostAddress.IsGitHubDotCom).ToArray();
-                connectionLabels = connections.Select(c => HostAddress.IsGitHubDotCom(c) ? "GitHub" : c.Host).ToArray();
+                connectionLabels = connections.Select(c => HostAddress.IsGitHubDotCom(c) ? "GitHub" : c.Host.ToUriString().Host).ToArray();
 
                 var connection = connections.First();
                 selectedConnection = 0;