review fixes

packages/core-api/src/apis/implementations/auth/google/GoogleAuth.ts

@@ -24,6 +24,7 @@ import {
   AccessTokenOptions,
   ProfileInfoApi,
   ProfileInfoOptions,
+  ProfileInfo,
 } from '../../../definitions/auth';
 import { OAuthRequestApi, AuthProvider } from '../../../definitions';
 import { SessionManager } from '../../../../lib/AuthSessionManager/types';
@@ -41,7 +42,7 @@ type CreateOptions = {
 };
 
 export type GoogleAuthResponse = {
-  profile: any;
+  profile: ProfileInfo;
   accessToken: string;
   idToken: string;
   scope: string;

plugins/auth-backend/src/providers/PassportStrategyHelper.ts

@@ -135,21 +135,31 @@ export const executeRefreshTokenStrategy = async (
           );
         }
 
-        anyStrategy.userProfile(
+        resolve({
           accessToken,
-          (error: Error, passportProfile: passport.Profile) => {
-            if (error) {
-              reject(error);
-            }
-
-            const profile = makeProfileInfo(passportProfile, params);
-            resolve({
-              accessToken,
-              params,
-              profile,
-            });
-          },
-        );
+          params,
+        });
+      },
+    );
+  });
+};
+
+export const executeFetchUserProfileStrategy = async (
+  providerstrategy: passport.Strategy,
+  accessToken: string,
+  params: any,
+): Promise<ProfileInfo> => {
+  return new Promise((resolve, reject) => {
+    const anyStrategy = providerstrategy as any;
+    anyStrategy.userProfile(
+      accessToken,
+      (error: Error, passportProfile: passport.Profile) => {
+        if (error) {
+          reject(error);
+        }
+
+        const profile = makeProfileInfo(passportProfile, params);
+        resolve(profile);
       },
     );
   });

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

@@ -21,6 +21,7 @@ import {
   executeRedirectStrategy,
   executeRefreshTokenStrategy,
   makeProfileInfo,
+  executeFetchUserProfileStrategy,
 } from '../PassportStrategyHelper';
 import {
   OAuthProviderHandlers,
@@ -81,12 +82,18 @@ export class GoogleAuthProvider implements OAuthProviderHandlers {
     refreshToken: string,
     scope: string,
   ): Promise<AuthInfoWithProfile> {
-    const { accessToken, params, profile } = await executeRefreshTokenStrategy(
+    const { accessToken, params } = await executeRefreshTokenStrategy(
       this._strategy,
       refreshToken,
       scope,
     );
 
+    const profile = await executeFetchUserProfileStrategy(
+      this._strategy,
+      accessToken,
+      params,
+    );
+
     return {
       accessToken,
       idToken: params.id_token,

plugins/auth-backend/src/providers/types.ts

@@ -87,5 +87,4 @@ export type ProfileInfo = {
 export type RefreshTokenResponse = {
   accessToken: string;
   params: any;
-  profile: ProfileInfo;
 };