We only need to avoid saving the keychain

Author: StanleyGoldman

src/GitHub.Api/Authentication/LoginManager.cs

@@ -84,9 +84,10 @@ public async Task<LoginResultData> Login(
                         throw new InvalidOperationException("Returned token is null or empty");
                     }
 
+                    keychain.SetToken(host, loginResultData.Token);
+
                     if (loginResultData.Code == LoginResultCodes.Success)
                     {
-                        keychain.SetToken(host, loginResultData.Token);
                         await keychain.Save(host);
                     }
 
@@ -109,7 +110,7 @@ public async Task<LoginResultData> ContinueLogin(LoginResultData loginResultData
             var host = loginResultData.Host;
             var keychainAdapter = keychain.Connect(host);
             var username = keychainAdapter.Credential.Username;
-            var password = loginResultData.Token;
+            var password = keychainAdapter.Credential.Token;
             try
             {
                 logger.Trace("2FA Continue");