@azure/msal-browser 4.27.0 → 5.0.0-beta.0

package/src/interaction_client/PlatformAuthInteractionClient.ts

@@ -6,20 +6,15 @@
 import {
     Logger,
     ICrypto,
-    PromptValue,
     AuthToken,
-    Constants,
     AccountEntity,
     AuthorityType,
     ScopeSet,
     TimeUtils,
-    AuthenticationScheme,
     UrlString,
-    OIDC_DEFAULT_SCOPES,
     PopTokenGenerator,
     SignedHttpRequestParameters,
     IPerformanceClient,
-    PerformanceEvents,
     IdTokenEntity,
     AccessTokenEntity,
     AuthError,
@@ -35,8 +30,17 @@ import {
     buildAccountToCache,
     InProgressPerformanceEvent,
     ServerTelemetryManager,
+    AccountEntityUtils,
+    Constants,
+    PerformanceEvents,
 } from "@azure/msal-common/browser";
-import { BaseInteractionClient } from "./BaseInteractionClient.js";
+import {
+    BaseInteractionClient,
+    getDiscoveredAuthority,
+    getRedirectUri,
+    initializeServerTelemetryManager,
+} from "./BaseInteractionClient.js";
+import * as BrowserPerformanceEvents from "../telemetry/BrowserPerformanceEvents.js";
 import { BrowserConfiguration } from "../config/Configuration.js";
 import { BrowserCacheManager } from "../cache/BrowserCacheManager.js";
 import { EventHandler } from "../event/EventHandler.js";
@@ -73,6 +77,7 @@ import { AuthenticationResult } from "../response/AuthenticationResult.js";
 import { base64Decode } from "../encode/Base64Decode.js";
 import { version } from "../packageMetadata.js";
 import { IPlatformAuthHandler } from "../broker/nativeBroker/IPlatformAuthHandler.js";
+import { HandleRedirectPromiseOptions } from "../request/HandleRedirectPromiseOptions.js";
 
 export class PlatformAuthInteractionClient extends BaseInteractionClient {
     protected apiId: ApiId;
@@ -94,7 +99,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         provider: IPlatformAuthHandler,
         accountId: string,
         nativeStorageImpl: BrowserCacheManager,
-        correlationId?: string
+        correlationId: string
     ) {
         super(
             config,
@@ -104,8 +109,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             eventHandler,
             navigationClient,
             performanceClient,
-            provider,
-            correlationId
+            correlationId,
+            provider
         );
         this.apiId = apiId;
         this.accountId = accountId;
@@ -119,8 +124,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             eventHandler,
             navigationClient,
             performanceClient,
-            provider,
-            correlationId
+            correlationId,
+            provider
         );
 
         const extensionName = this.platformAuthProvider.getExtensionName();
@@ -153,21 +158,24 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         request: PopupRequest | SilentRequest | SsoSilentRequest,
         cacheLookupPolicy?: CacheLookupPolicy
     ): Promise<AuthenticationResult> {
-        this.performanceClient.addQueueMeasurement(
-            PerformanceEvents.NativeInteractionClientAcquireToken,
+        this.logger.trace(
+            "NativeInteractionClient - acquireToken called.",
             this.correlationId
         );
-        this.logger.trace("NativeInteractionClient - acquireToken called.");
 
         // start the perf measurement
         const nativeATMeasurement = this.performanceClient.startMeasurement(
-            PerformanceEvents.NativeInteractionClientAcquireToken,
-            this.correlationId
+            BrowserPerformanceEvents.NativeInteractionClientAcquireToken,
+            request.correlationId
         );
         const reqTimestamp = TimeUtils.nowSeconds();
 
-        const serverTelemetryManager = this.initializeServerTelemetryManager(
-            this.apiId
+        const serverTelemetryManager = initializeServerTelemetryManager(
+            this.apiId,
+            this.config.auth.clientId,
+            this.correlationId,
+            this.browserStorage,
+            this.logger
         );
 
         try {
@@ -189,17 +197,15 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             } catch (e) {
                 if (cacheLookupPolicy === CacheLookupPolicy.AccessToken) {
                     this.logger.info(
-                        "MSAL internal Cache does not contain tokens, return error as per cache policy"
+                        "MSAL internal Cache does not contain tokens, return error as per cache policy",
+                        this.correlationId
                     );
-                    nativeATMeasurement.end({
-                        success: false,
-                        brokerErrorCode: "cache_request_failed",
-                    });
                     throw e;
                 }
                 // continue with a native call for any and all errors
                 this.logger.info(
-                    "MSAL internal Cache does not contain tokens, proceed to make a native call"
+                    "MSAL internal Cache does not contain tokens, proceed to make a native call",
+                    this.correlationId
                 );
             }
 
@@ -225,6 +231,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
                         success: false,
                         errorCode: error.errorCode,
                         subErrorCode: error.subError,
+                        isNativeBroker: true,
                     });
                     throw error;
                 });
@@ -232,9 +239,6 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             if (e instanceof NativeAuthError) {
                 serverTelemetryManager.setNativeBrokerErrorCode(e.errorCode);
             }
-            nativeATMeasurement.end({
-                success: false,
-            });
             throw e;
         }
     }
@@ -270,7 +274,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
     ): Promise<AuthenticationResult> {
         if (!nativeAccountId) {
             this.logger.warning(
-                "NativeInteractionClient:acquireTokensFromCache - No nativeAccountId provided"
+                "NativeInteractionClient:acquireTokensFromCache - No nativeAccountId provided",
+                this.correlationId
             );
             throw createClientAuthError(ClientAuthErrorCodes.noAccountFound);
         }
@@ -279,7 +284,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             {
                 nativeAccountId,
             },
-            this.correlationId
+            request.correlationId
         );
 
         if (!account) {
@@ -315,29 +320,34 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
      * Acquires a token from native platform then redirects to the redirectUri instead of returning the response
      * @param {RedirectRequest} request
      * @param {InProgressPerformanceEvent} rootMeasurement
+     * @param {HandleRedirectPromiseOptions} options
      */
     async acquireTokenRedirect(
         request: RedirectRequest,
-        rootMeasurement: InProgressPerformanceEvent
+        rootMeasurement: InProgressPerformanceEvent,
+        options?: HandleRedirectPromiseOptions
     ): Promise<void> {
         this.logger.trace(
-            "NativeInteractionClient - acquireTokenRedirect called."
+            "NativeInteractionClient - acquireTokenRedirect called.",
+            this.correlationId
         );
 
-        const { ...remainingParameters } = request;
-        delete remainingParameters.onRedirectNavigate;
-
-        const nativeRequest = await this.initializeNativeRequest(
-            remainingParameters
-        );
+        const nativeRequest = await this.initializeNativeRequest(request);
+        const navigateToLoginRequestUrl =
+            options?.navigateToLoginRequestUrl ?? true;
 
         try {
             await this.platformAuthProvider.sendMessage(nativeRequest);
         } catch (e) {
             // Only throw fatal errors here to allow application to fallback to regular redirect. Otherwise proceed and the error will be thrown in handleRedirectPromise
             if (e instanceof NativeAuthError) {
-                const serverTelemetryManager =
-                    this.initializeServerTelemetryManager(this.apiId);
+                const serverTelemetryManager = initializeServerTelemetryManager(
+                    this.apiId,
+                    this.config.auth.clientId,
+                    this.correlationId,
+                    this.browserStorage,
+                    this.logger
+                );
                 serverTelemetryManager.setNativeBrokerErrorCode(e.errorCode);
                 if (isFatalNativeAuthError(e)) {
                     throw e;
@@ -355,9 +365,14 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             timeout: this.config.system.redirectNavigationTimeout,
             noHistory: false,
         };
-        const redirectUri = this.config.auth.navigateToLoginRequestUrl
+        const redirectUri = navigateToLoginRequestUrl
             ? window.location.href
-            : this.getRedirectUri(request.redirectUri);
+            : getRedirectUri(
+                  request.redirectUri,
+                  this.config.auth.redirectUri,
+                  this.logger,
+                  this.correlationId
+              );
         rootMeasurement.end({ success: true });
         await this.navigationClient.navigateExternal(
             redirectUri,
@@ -375,11 +390,13 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         correlationId?: string
     ): Promise<AuthenticationResult | null> {
         this.logger.trace(
-            "NativeInteractionClient - handleRedirectPromise called."
+            "NativeInteractionClient - handleRedirectPromise called.",
+            this.correlationId
         );
         if (!this.browserStorage.isInteractionInProgress(true)) {
             this.logger.info(
-                "handleRedirectPromise called but there is no interaction in progress, returning null."
+                "handleRedirectPromise called but there is no interaction in progress, returning null.",
+                this.correlationId
             );
             return null;
         }
@@ -388,7 +405,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         const cachedRequest = this.browserStorage.getCachedNativeRequest();
         if (!cachedRequest) {
             this.logger.verbose(
-                "NativeInteractionClient - handleRedirectPromise called but there is no cached request, returning null."
+                "NativeInteractionClient - handleRedirectPromise called but there is no cached request, returning null.",
+                this.correlationId
             );
             if (performanceClient && correlationId) {
                 performanceClient?.addFields(
@@ -402,7 +420,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         const { prompt, ...request } = cachedRequest;
         if (prompt) {
             this.logger.verbose(
-                "NativeInteractionClient - handleRedirectPromise called and prompt was included in the original request, removing prompt from cached request to prevent second interaction with native broker window."
+                "NativeInteractionClient - handleRedirectPromise called and prompt was included in the original request, removing prompt from cached request to prevent second interaction with native broker window.",
+                this.correlationId
             );
         }
 
@@ -416,7 +435,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
 
         try {
             this.logger.verbose(
-                "NativeInteractionClient - handleRedirectPromise sending message to native broker."
+                "NativeInteractionClient - handleRedirectPromise sending message to native broker.",
+                this.correlationId
             );
             const response: PlatformAuthResponse =
                 await this.platformAuthProvider.sendMessage(request);
@@ -426,15 +446,14 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
                 reqTimestamp
             );
 
-            const serverTelemetryManager =
-                this.initializeServerTelemetryManager(this.apiId);
+            const serverTelemetryManager = initializeServerTelemetryManager(
+                this.apiId,
+                this.config.auth.clientId,
+                this.correlationId,
+                this.browserStorage,
+                this.logger
+            );
             serverTelemetryManager.clearNativeBrokerErrorCode();
-            if (performanceClient && this.correlationId) {
-                this.performanceClient.addFields(
-                    { isNativeBroker: true },
-                    this.correlationId
-                );
-            }
             return authResult;
         } catch (e) {
             throw e;
@@ -446,7 +465,10 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
      * @param request
      */
     logout(): Promise<void> {
-        this.logger.trace("NativeInteractionClient - logout called.");
+        this.logger.trace(
+            "NativeInteractionClient - logout called.",
+            this.correlationId
+        );
         return Promise.reject("Logout not implemented yet");
     }
 
@@ -462,7 +484,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         reqTimestamp: number
     ): Promise<AuthenticationResult> {
         this.logger.trace(
-            "NativeInteractionClient - handleNativeResponse called."
+            "NativeInteractionClient - handleNativeResponse called.",
+            this.correlationId
         );
 
         // generate identifiers
@@ -490,7 +513,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             response.account.id !== request.accountId
         ) {
             this.logger.info(
-                "handleNativeServerResponse: Double broker flow detected, ignoring accountId mismatch"
+                "handleNativeServerResponse: Double broker flow detected, ignoring accountId mismatch",
+                this.correlationId
             );
         } else if (
             homeAccountIdentifier !== cachedhomeAccountId &&
@@ -501,9 +525,14 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         }
 
         // Get the preferred_cache domain for the given authority
-        const authority = await this.getDiscoveredAuthority({
-            requestAuthority: request.authority,
-        });
+        const authority = await getDiscoveredAuthority(
+            this.config,
+            this.correlationId,
+            this.performanceClient,
+            this.browserStorage,
+            this.logger,
+            request.authority
+        );
 
         const baseAccount = buildAccountToCache(
             this.browserStorage,
@@ -516,8 +545,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             undefined, // environment
             idTokenClaims.tid,
             undefined, // auth code payload
-            response.account.id,
-            this.logger
+            response.account.id
         );
 
         // Ensure expires_in is in number format
@@ -534,11 +562,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         );
 
         // cache accounts and tokens in the appropriate storage
-        await this.cacheAccount(
-            baseAccount,
-            this.correlationId,
-            AuthToken.isKmsi(idTokenClaims)
-        );
+        await this.cacheAccount(baseAccount, AuthToken.isKmsi(idTokenClaims));
         await this.cacheNativeTokens(
             response,
             request,
@@ -563,11 +587,12 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         idTokenClaims: TokenClaims
     ): string {
         // Save account in browser storage
-        const homeAccountIdentifier = AccountEntity.generateHomeAccountId(
-            response.client_info || Constants.EMPTY_STRING,
+        const homeAccountIdentifier = AccountEntityUtils.generateHomeAccountId(
+            response.client_info || "",
             AuthorityType.Default,
             this.logger,
             this.browserCrypto,
+            this.correlationId,
             idTokenClaims
         );
 
@@ -596,7 +621,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         request: PlatformAuthRequest
     ): Promise<string> {
         if (
-            request.tokenType === AuthenticationScheme.POP &&
+            request.tokenType === Constants.AuthenticationScheme.POP &&
             request.signPopToken
         ) {
             /**
@@ -607,20 +632,23 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             // Check if native layer returned an SHR token
             if (response.shr) {
                 this.logger.trace(
-                    "handleNativeServerResponse: SHR is enabled in native layer"
+                    "handleNativeServerResponse: SHR is enabled in native layer",
+                    this.correlationId
                 );
                 return response.shr;
             }
 
             // Generate SHR in msal js if WAM does not compute it when POP is enabled
             const popTokenGenerator: PopTokenGenerator = new PopTokenGenerator(
-                this.browserCrypto
+                this.browserCrypto,
+                this.performanceClient
             );
             const shrParameters: SignedHttpRequestParameters = {
                 resourceRequestMethod: request.resourceRequestMethod,
                 resourceRequestUri: request.resourceRequestUri,
                 shrClaims: request.shrClaims,
                 shrNonce: request.shrNonce,
+                correlationId: this.correlationId,
             };
 
             /**
@@ -674,14 +702,11 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             accountProperties["UID"] ||
             idTokenClaims.oid ||
             idTokenClaims.sub ||
-            Constants.EMPTY_STRING;
-        const tid =
-            accountProperties["TenantId"] ||
-            idTokenClaims.tid ||
-            Constants.EMPTY_STRING;
+            "";
+        const tid = accountProperties["TenantId"] || idTokenClaims.tid || "";
 
         const accountInfo: AccountInfo | null = updateAccountTenantProfileData(
-            AccountEntity.getAccountInfo(accountEntity),
+            AccountEntityUtils.getAccountInfo(accountEntity),
             undefined, // tenantProfile optional
             idTokenClaims,
             response.id_token
@@ -701,9 +726,9 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             request
         );
         const tokenType =
-            request.tokenType === AuthenticationScheme.POP
-                ? AuthenticationScheme.POP
-                : AuthenticationScheme.BEARER;
+            request.tokenType === Constants.AuthenticationScheme.POP
+                ? Constants.AuthenticationScheme.POP
+                : Constants.AuthenticationScheme.BEARER;
 
         const result: AuthenticationResult = {
             authority: authority,
@@ -722,7 +747,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             tokenType: tokenType,
             correlationId: this.correlationId,
             state: response.state,
-            fromNativeBroker: true,
+            fromPlatformBroker: true,
         };
 
         return result;
@@ -734,7 +759,6 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
      */
     async cacheAccount(
         accountEntity: AccountEntity,
-        correlationId: string,
         kmsi: boolean
     ): Promise<void> {
         // Store the account info and hence `nativeAccountId` in browser cache
@@ -745,8 +769,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         );
         // Remove any existing cached tokens for this account in browser storage
         this.browserStorage.removeAccountContext(
-            AccountEntity.getAccountInfo(accountEntity),
-            correlationId
+            AccountEntityUtils.getAccountInfo(accountEntity),
+            this.correlationId
         );
     }
 
@@ -780,7 +804,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
 
         // cache accessToken in inmemory storage
         const expiresIn: number =
-            request.tokenType === AuthenticationScheme.POP
+            request.tokenType === Constants.AuthenticationScheme.POP
                 ? Constants.SHR_NONCE_VALIDITY
                 : (typeof response.expires_in === "string"
                       ? parseInt(response.expires_in, 10)
@@ -803,7 +827,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
                 0,
                 base64Decode,
                 undefined,
-                request.tokenType as AuthenticationScheme,
+                request.tokenType as Constants.AuthenticationScheme,
                 undefined,
                 request.keyId
             );
@@ -825,7 +849,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         tokenType: string,
         expiresIn: string | number | undefined
     ): number {
-        return tokenType === AuthenticationScheme.POP
+        return tokenType === Constants.AuthenticationScheme.POP
             ? Constants.SHR_NONCE_VALIDITY
             : (typeof expiresIn === "string"
                   ? parseInt(expiresIn, 10)
@@ -877,7 +901,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
                 return JSON.parse(matsResponse);
             } catch (e) {
                 this.logger.error(
-                    "NativeInteractionClient - Error parsing MATS telemetry, returning null instead"
+                    "NativeInteractionClient - Error parsing MATS telemetry, returning null instead",
+                    this.correlationId
                 );
             }
         }
@@ -893,7 +918,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
     protected isResponseFromCache(mats: MATS): boolean {
         if (typeof mats.is_cached === "undefined") {
             this.logger.verbose(
-                "NativeInteractionClient - MATS telemetry does not contain field indicating if response was served from cache. Returning false."
+                "NativeInteractionClient - MATS telemetry does not contain field indicating if response was served from cache. Returning false.",
+                this.correlationId
             );
             return false;
         }
@@ -909,7 +935,8 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         request: PopupRequest | SsoSilentRequest
     ): Promise<PlatformAuthRequest> {
         this.logger.trace(
-            "NativeInteractionClient - initializeNativeRequest called"
+            "NativeInteractionClient - initializeNativeRequest called",
+            this.correlationId
         );
 
         const canonicalAuthority = await this.getCanonicalAuthority(request);
@@ -917,7 +944,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         // scopes are expected to be received by the native broker as "scope" and will be added to the request below. Other properties that should be dropped from the request to the native broker can be included in the object destructuring here.
         const { scopes, ...remainingProperties } = request;
         const scopeSet = new ScopeSet(scopes || []);
-        scopeSet.appendScopes(OIDC_DEFAULT_SCOPES);
+        scopeSet.appendScopes(Constants.OIDC_DEFAULT_SCOPES);
 
         const validatedRequest: PlatformAuthRequest = {
             ...remainingProperties,
@@ -925,14 +952,18 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             clientId: this.config.auth.clientId,
             authority: canonicalAuthority.urlString,
             scope: scopeSet.printScopes(),
-            redirectUri: this.getRedirectUri(request.redirectUri),
+            redirectUri: getRedirectUri(
+                request.redirectUri,
+                this.config.auth.redirectUri,
+                this.logger,
+                this.correlationId
+            ),
             prompt: this.getPrompt(request.prompt),
             correlationId: this.correlationId,
             tokenType: request.authenticationScheme,
             windowTitleSubstring: document.title,
             extraParameters: {
-                ...request.extraQueryParameters,
-                ...request.tokenQueryParameters,
+                ...request.extraParameters,
             },
             extendedExpiryToken: false, // Make this configurable?
             keyId: request.popKid,
@@ -951,16 +982,22 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         validatedRequest.extraParameters.telemetry =
             PlatformAuthConstants.MATS_TELEMETRY;
 
-        if (request.authenticationScheme === AuthenticationScheme.POP) {
+        if (
+            request.authenticationScheme === Constants.AuthenticationScheme.POP
+        ) {
             // add POP request type
             const shrParameters: SignedHttpRequestParameters = {
                 resourceRequestUri: request.resourceRequestUri,
                 resourceRequestMethod: request.resourceRequestMethod,
                 shrClaims: request.shrClaims,
                 shrNonce: request.shrNonce,
+                correlationId: this.correlationId,
             };
 
-            const popTokenGenerator = new PopTokenGenerator(this.browserCrypto);
+            const popTokenGenerator = new PopTokenGenerator(
+                this.browserCrypto,
+                this.performanceClient
+            );
 
             // generate reqCnf if not provided in the request
             let reqCnfData;
@@ -996,13 +1033,21 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         const requestAuthority =
             request.authority || this.config.auth.authority;
 
-        if (request.account) {
+        const { azureCloudOptions, account } = request;
+
+        if (account) {
             // validate authority
-            await this.getDiscoveredAuthority({
+            await getDiscoveredAuthority(
+                this.config,
+                this.correlationId,
+                this.performanceClient,
+                this.browserStorage,
+                this.logger,
                 requestAuthority,
-                requestAzureCloudOptions: request.azureCloudOptions,
-                account: request.account,
-            });
+                azureCloudOptions,
+                undefined, // requestExtraQueryParameters
+                account
+            );
         }
 
         const canonicalAuthority = new UrlString(requestAuthority);
@@ -1016,9 +1061,10 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             case ApiId.ssoSilent:
             case ApiId.acquireTokenSilent_silentFlow:
                 this.logger.trace(
-                    "initializeNativeRequest: silent request sets prompt to none"
+                    "initializeNativeRequest: silent request sets prompt to none",
+                    this.correlationId
                 );
-                return PromptValue.NONE;
+                return Constants.PromptValue.NONE;
             default:
                 break;
         }
@@ -1026,24 +1072,26 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
         // Prompt not provided, request may proceed and native broker decides if it needs to prompt
         if (!prompt) {
             this.logger.trace(
-                "initializeNativeRequest: prompt was not provided"
+                "initializeNativeRequest: prompt was not provided",
+                this.correlationId
             );
             return undefined;
         }
 
         // If request is interactive, check if prompt provided is allowed to go directly to native broker
         switch (prompt) {
-            case PromptValue.NONE:
-            case PromptValue.CONSENT:
-            case PromptValue.LOGIN:
-            case PromptValue.SELECT_ACCOUNT:
+            case Constants.PromptValue.NONE:
+            case Constants.PromptValue.CONSENT:
+            case Constants.PromptValue.LOGIN:
                 this.logger.trace(
-                    "initializeNativeRequest: prompt is compatible with native flow"
+                    "initializeNativeRequest: prompt is compatible with native flow",
+                    this.correlationId
                 );
                 return prompt;
             default:
                 this.logger.trace(
-                    `initializeNativeRequest: prompt = ${prompt} is not compatible with native flow`
+                    `initializeNativeRequest: prompt = '${prompt}' is not compatible with native flow`,
+                    this.correlationId
                 );
                 throw createBrowserAuthError(
                     BrowserAuthErrorCodes.nativePromptNotSupported
@@ -1096,7 +1144,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
                 embeddedClientId: child_client_id,
                 embeddedRedirectUri: child_redirect_uri,
             },
-            this.correlationId
+            request.correlationId
         );
     }
 }