Merge pull request #4629 from backstage/rugvip/auops

auth-backend: fix for refresh token being lost during microsoft login

.changeset/breezy-jobs-brake.md

@@ -0,0 +1,5 @@
+---
+'@backstage/plugin-auth-backend': patch
+---
+
+Fix for refresh token being lost during Microsoft login.

plugins/auth-backend/src/providers/github/provider.ts

@@ -54,12 +54,12 @@ export class GithubAuthProvider implements OAuthHandlers {
       },
       (
         accessToken: any,
-        refreshToken: any,
+        _refreshToken: any,
         params: any,
         fullProfile: any,
         done: PassportDoneCallback<OAuthResult>,
       ) => {
-        done(undefined, { fullProfile, params, accessToken, refreshToken });
+        done(undefined, { fullProfile, params, accessToken });
       },
     );
   }

plugins/auth-backend/src/providers/gitlab/provider.ts

@@ -51,12 +51,12 @@ export class GitlabAuthProvider implements OAuthHandlers {
       },
       (
         accessToken: any,
-        refreshToken: any,
+        _refreshToken: any,
         params: any,
         fullProfile: any,
         done: PassportDoneCallback<OAuthResult>,
       ) => {
-        done(undefined, { fullProfile, params, accessToken, refreshToken });
+        done(undefined, { fullProfile, params, accessToken });
       },
     );
   }

plugins/auth-backend/src/providers/microsoft/provider.ts

@@ -72,7 +72,7 @@ export class MicrosoftAuthProvider implements OAuthHandlers {
         fullProfile: passport.Profile,
         done: PassportDoneCallback<OAuthResult, PrivateInfo>,
       ) => {
-        done(undefined, { fullProfile, accessToken, refreshToken, params });
+        done(undefined, { fullProfile, accessToken, params }, { refreshToken });
       },
     );
   }