GenericHostProvider: immediately store refresh token

hickford · hickford · commit 39ba57ef58ee · 2026-01-10T19:40:35.000Z
Suggested-by: Marc Becker
diff --git a/src/shared/Core/GenericHostProvider.cs b/src/shared/Core/GenericHostProvider.cs
@@ -150,6 +150,13 @@ private async Task<ICredential> GetOAuthAccessToken(Uri remoteUri, string userNa
                 {
                     var refreshResult = await client.GetTokenByRefreshTokenAsync(refreshToken, CancellationToken.None);
 
+                    // Store new refresh token if we have been given one
+                    if (!string.IsNullOrWhiteSpace(refreshResult.RefreshToken))
+                    {
+                        var refreshService = GetRefreshTokenServiceName(remoteUri);
+                        Context.CredentialStore.AddOrUpdate(refreshService, oauthUser, refreshResult.RefreshToken);
+                    }
+
                     // Return the new access token
                     return new GitCredential(refreshResult, oauthUser);
                 }

Original file line number	Diff line number	Diff line change
`@@ -150,6 +150,13 @@ private async Task<ICredential> GetOAuthAccessToken(Uri remoteUri, string userNa`
`150`	`150`	`{`
`151`	`151`	`var refreshResult = await client.GetTokenByRefreshTokenAsync(refreshToken, CancellationToken.None);`
`152`	`152`
	`153`	`+ // Store new refresh token if we have been given one`
	`154`	`+ if (!string.IsNullOrWhiteSpace(refreshResult.RefreshToken))`
	`155`	`+ {`
	`156`	`+ var refreshService = GetRefreshTokenServiceName(remoteUri);`
	`157`	`+ Context.CredentialStore.AddOrUpdate(refreshService, oauthUser, refreshResult.RefreshToken);`
	`158`	`+ }`
	`159`	`+`
`153`	`160`	`// Return the new access token`
`154`	`161`	`return new GitCredential(refreshResult, oauthUser);`
`155`	`162`	`}`