@azure/msal-common 15.12.0 → 16.0.0-alpha.0

package/src/error/InteractionRequiredAuthError.ts

@@ -3,7 +3,6 @@
  * Licensed under the MIT License.
  */
 
-import { Constants } from "../utils/Constants.js";
 import { AuthError } from "./AuthError.js";
 import * as InteractionRequiredAuthErrorCodes from "./InteractionRequiredAuthErrorCodes.js";
 export { InteractionRequiredAuthErrorCodes };
@@ -26,46 +25,9 @@ export const InteractionRequiredAuthSubErrorMessage = [
     "user_password_expired",
     "consent_required",
     "bad_token",
+    "ux_not_allowed",
 ];
 
-const InteractionRequiredAuthErrorMessages = {
-    [InteractionRequiredAuthErrorCodes.noTokensFound]:
-        "No refresh token found in the cache. Please sign-in.",
-    [InteractionRequiredAuthErrorCodes.nativeAccountUnavailable]:
-        "The requested account is not available in the native broker. It may have been deleted or logged out. Please sign-in again using an interactive API.",
-    [InteractionRequiredAuthErrorCodes.refreshTokenExpired]:
-        "Refresh token has expired.",
-    [InteractionRequiredAuthErrorCodes.badToken]:
-        "Identity provider returned bad_token due to an expired or invalid refresh token. Please invoke an interactive API to resolve.",
-    [InteractionRequiredAuthErrorCodes.uxNotAllowed]:
-        "`canShowUI` flag in Edge was set to false. User interaction required on web page. Please invoke an interactive API to resolve.",
-};
-
-/**
- * Interaction required errors defined by the SDK
- * @deprecated Use InteractionRequiredAuthErrorCodes instead
- */
-export const InteractionRequiredAuthErrorMessage = {
-    noTokensFoundError: {
-        code: InteractionRequiredAuthErrorCodes.noTokensFound,
-        desc: InteractionRequiredAuthErrorMessages[
-            InteractionRequiredAuthErrorCodes.noTokensFound
-        ],
-    },
-    native_account_unavailable: {
-        code: InteractionRequiredAuthErrorCodes.nativeAccountUnavailable,
-        desc: InteractionRequiredAuthErrorMessages[
-            InteractionRequiredAuthErrorCodes.nativeAccountUnavailable
-        ],
-    },
-    bad_token: {
-        code: InteractionRequiredAuthErrorCodes.badToken,
-        desc: InteractionRequiredAuthErrorMessages[
-            InteractionRequiredAuthErrorCodes.badToken
-        ],
-    },
-};
-
 /**
  * Error thrown when user interaction is required.
  */
@@ -109,10 +71,10 @@ export class InteractionRequiredAuthError extends AuthError {
         super(errorCode, errorMessage, subError);
         Object.setPrototypeOf(this, InteractionRequiredAuthError.prototype);
 
-        this.timestamp = timestamp || Constants.EMPTY_STRING;
-        this.traceId = traceId || Constants.EMPTY_STRING;
-        this.correlationId = correlationId || Constants.EMPTY_STRING;
-        this.claims = claims || Constants.EMPTY_STRING;
+        this.timestamp = timestamp || "";
+        this.traceId = traceId || "";
+        this.correlationId = correlationId || "";
+        this.claims = claims || "";
         this.name = "InteractionRequiredAuthError";
         this.errorNo = errorNo;
     }
@@ -152,10 +114,8 @@ export function isInteractionRequiredError(
  * Creates an InteractionRequiredAuthError
  */
 export function createInteractionRequiredAuthError(
-    errorCode: string
+    errorCode: string,
+    errorMessage?: string
 ): InteractionRequiredAuthError {
-    return new InteractionRequiredAuthError(
-        errorCode,
-        InteractionRequiredAuthErrorMessages[errorCode]
-    );
+    return new InteractionRequiredAuthError(errorCode, errorMessage);
 }

package/src/error/JoseHeaderError.ts

@@ -7,13 +7,6 @@ import { AuthError } from "./AuthError.js";
 import * as JoseHeaderErrorCodes from "./JoseHeaderErrorCodes.js";
 export { JoseHeaderErrorCodes };
 
-export const JoseHeaderErrorMessages = {
-    [JoseHeaderErrorCodes.missingKidError]:
-        "The JOSE Header for the requested JWT, JWS or JWK object requires a keyId to be configured as the 'kid' header claim. No 'kid' value was provided.",
-    [JoseHeaderErrorCodes.missingAlgError]:
-        "The JOSE Header for the requested JWT, JWS or JWK object requires an algorithm to be specified as the 'alg' header claim. No 'alg' value was provided.",
-};
-
 /**
  * Error thrown when there is an error in the client code running on the browser.
  */
@@ -28,5 +21,5 @@ export class JoseHeaderError extends AuthError {
 
 /** Returns JoseHeaderError object */
 export function createJoseHeaderError(code: string): JoseHeaderError {
-    return new JoseHeaderError(code, JoseHeaderErrorMessages[code]);
+    return new JoseHeaderError(code);
 }

package/src/exports-browser-only.ts

@@ -10,20 +10,16 @@ export {
     IPerformanceClient,
     PerformanceCallbackFunction,
     InProgressPerformanceEvent,
-    QueueMeasurement,
 } from "./telemetry/performance/IPerformanceClient.js";
 export {
     IntFields,
     PerformanceEvent,
-    PerformanceEvents,
     PerformanceEventStatus,
     SubMeasurement,
 } from "./telemetry/performance/PerformanceEvent.js";
+export * as PerformanceEvents from "./telemetry/performance/PerformanceEvents.js";
 export { IPerformanceMeasurement } from "./telemetry/performance/IPerformanceMeasurement.js";
-export {
-    PerformanceClient,
-    PreQueueEvent,
-} from "./telemetry/performance/PerformanceClient.js";
+export { PerformanceClient } from "./telemetry/performance/PerformanceClient.js";
 export { StubPerformanceClient } from "./telemetry/performance/StubPerformanceClient.js";
 
 export { PopTokenGenerator } from "./crypto/PopTokenGenerator.js";

package/src/exports-common.ts

@@ -9,6 +9,7 @@ import * as CacheHelpers from "./cache/utils/CacheHelpers.js";
 import * as TimeUtils from "./utils/TimeUtils.js";
 import * as UrlUtils from "./utils/UrlUtils.js";
 import * as AADServerParamKeys from "./constants/AADServerParamKeys.js";
+import * as AccountEntityUtils from "./cache/utils/AccountEntityUtils.js";
 
 export { AuthToken };
 export { AuthorityFactory };
@@ -16,6 +17,7 @@ export { CacheHelpers };
 export { TimeUtils };
 export { UrlUtils };
 export { AADServerParamKeys };
+export { AccountEntityUtils };
 
 export { AuthorizationCodeClient } from "./client/AuthorizationCodeClient.js";
 export { RefreshTokenClient } from "./client/RefreshTokenClient.js";
@@ -25,7 +27,6 @@ export {
     AuthOptions,
     SystemOptions,
     LoggerOptions,
-    CacheOptions,
     DEFAULT_SYSTEM_OPTIONS,
     AzureCloudOptions,
     ApplicationTelemetry,
@@ -131,12 +132,10 @@ export { ILoggerCallback, LogLevel, Logger } from "./logger/Logger.js";
 export {
     InteractionRequiredAuthError,
     InteractionRequiredAuthErrorCodes,
-    InteractionRequiredAuthErrorMessage,
     createInteractionRequiredAuthError,
 } from "./error/InteractionRequiredAuthError.js";
 export {
     AuthError,
-    AuthErrorMessage,
     AuthErrorCodes,
     createAuthError,
 } from "./error/AuthError.js";
@@ -149,45 +148,16 @@ export {
 } from "./error/CacheError.js";
 export {
     ClientAuthError,
-    ClientAuthErrorMessage,
     ClientAuthErrorCodes,
     createClientAuthError,
 } from "./error/ClientAuthError.js";
 export {
     ClientConfigurationError,
-    ClientConfigurationErrorMessage,
     ClientConfigurationErrorCodes,
     createClientConfigurationError,
 } from "./error/ClientConfigurationError.js";
-export {
-    Constants,
-    OIDC_DEFAULT_SCOPES,
-    PromptValue,
-    PersistentCacheKeys,
-    OAuthResponseType,
-    ServerResponseType,
-    ResponseMode,
-    CacheOutcome,
-    CredentialType,
-    CacheType,
-    CacheAccountType,
-    AuthenticationScheme,
-    CodeChallengeMethodValues,
-    PasswordGrantConstants,
-    ThrottlingConstants,
-    ClaimsRequestKeys,
-    HeaderNames,
-    Errors,
-    THE_FAMILY_ID,
-    ONE_DAY_IN_MS,
-    GrantType,
-    AADAuthorityConstants,
-    HttpStatus,
-    HttpMethod,
-    DEFAULT_TOKEN_RENEWAL_OFFSET_SEC,
-    JsonWebTokenTypes,
-    EncodingTypes,
-} from "./utils/Constants.js";
+export * as Constants from "./utils/Constants.js";
+
 export { StringUtils } from "./utils/StringUtils.js";
 export { StringDict } from "./utils/MsalTypes.js";
 export {

package/src/exports-node-only.ts

@@ -16,10 +16,6 @@ export { INativeBrokerPlugin } from "./broker/nativeBroker/INativeBrokerPlugin.j
 export { ICachePlugin } from "./cache/interface/ICachePlugin.js";
 export { TokenCacheContext } from "./cache/persistence/TokenCacheContext.js";
 export { ISerializableTokenCache } from "./cache/interface/ISerializableTokenCache.js";
-export { CommonClientCredentialRequest } from "./request/CommonClientCredentialRequest.js";
-export { CommonOnBehalfOfRequest } from "./request/CommonOnBehalfOfRequest.js";
-export { CommonDeviceCodeRequest } from "./request/CommonDeviceCodeRequest.js";
-export { CommonUsernamePasswordRequest } from "./request/CommonUsernamePasswordRequest.js";
 export { NativeRequest } from "./request/NativeRequest.js";
 export { NativeSignOutRequest } from "./request/NativeSignOutRequest.js";
 export {
@@ -33,3 +29,4 @@ export {
 } from "./response/DeviceCodeResponse.js";
 export { getClientAssertion } from "./utils/ClientAssertionUtils.js";
 export { IGuidGenerator } from "./crypto/IGuidGenerator.js";
+export { StubPerformanceClient } from "./telemetry/performance/StubPerformanceClient.js";

package/src/logger/Logger.ts

@@ -4,7 +4,6 @@
  */
 
 import type { LoggerOptions } from "../config/ClientConfiguration.js";
-import { Constants } from "../utils/Constants.js";
 
 /**
  * Options for logger messages.
@@ -13,7 +12,7 @@ export type LoggerMessageOptions = {
     logLevel: LogLevel;
     containsPii?: boolean;
     context?: string;
-    correlationId?: string;
+    correlationId: string;
 };
 
 /**
@@ -38,9 +37,6 @@ export interface ILoggerCallback {
  * Class which facilitates logging of messages to a specific place.
  */
 export class Logger {
-    // Correlation ID for request, usually set by user.
-    private correlationId: string;
-
     // Current log level, defaults to info.
     private level: LogLevel = LogLevel.Info;
 
@@ -73,10 +69,8 @@ export class Logger {
             typeof setLoggerOptions.logLevel === "number"
                 ? setLoggerOptions.logLevel
                 : LogLevel.Info;
-        this.correlationId =
-            setLoggerOptions.correlationId || Constants.EMPTY_STRING;
-        this.packageName = packageName || Constants.EMPTY_STRING;
-        this.packageVersion = packageVersion || Constants.EMPTY_STRING;
+        this.packageName = packageName || "";
+        this.packageVersion = packageVersion || "";
     }
 
     private static createDefaultLoggerOptions(): LoggerOptions {
@@ -92,17 +86,12 @@ export class Logger {
     /**
      * Create new Logger with existing configurations.
      */
-    public clone(
-        packageName: string,
-        packageVersion: string,
-        correlationId?: string
-    ): Logger {
+    public clone(packageName: string, packageVersion: string): Logger {
         return new Logger(
             {
                 loggerCallback: this.localCallback,
                 piiLoggingEnabled: this.piiLoggingEnabled,
                 logLevel: this.level,
-                correlationId: correlationId || this.correlationId,
             },
             packageName,
             packageVersion
@@ -125,14 +114,12 @@ export class Logger {
         const timestamp = new Date().toUTCString();
 
         // Add correlationId to logs if set, correlationId provided on log messages take precedence
-        const logHeader = `[${timestamp}] : [${
-            options.correlationId || this.correlationId || ""
-        }]`;
+        const logHeader = `[${timestamp}] : [${options.correlationId}]`;
 
         const log = `${logHeader} : ${this.packageName}@${
             this.packageVersion
         } : ${LogLevel[options.logLevel]} - ${logMessage}`;
-        // debug(`msal:${LogLevel[options.logLevel]}${options.containsPii ? "-Pii": Constants.EMPTY_STRING}${options.context ? `:${options.context}` : Constants.EMPTY_STRING}`)(logMessage);
+        // debug(`msal:${LogLevel[options.logLevel]}${options.containsPii ? "-Pii": Constants.""}${options.context ? `:${options.context}` : Constants.""}`)(logMessage);
         this.executeCallback(
             options.logLevel,
             log,
@@ -156,110 +143,110 @@ export class Logger {
     /**
      * Logs error messages.
      */
-    error(message: string, correlationId?: string): void {
+    error(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Error,
             containsPii: false,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs error messages with PII.
      */
-    errorPii(message: string, correlationId?: string): void {
+    errorPii(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Error,
             containsPii: true,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs warning messages.
      */
-    warning(message: string, correlationId?: string): void {
+    warning(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Warning,
             containsPii: false,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs warning messages with PII.
      */
-    warningPii(message: string, correlationId?: string): void {
+    warningPii(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Warning,
             containsPii: true,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs info messages.
      */
-    info(message: string, correlationId?: string): void {
+    info(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Info,
             containsPii: false,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs info messages with PII.
      */
-    infoPii(message: string, correlationId?: string): void {
+    infoPii(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Info,
             containsPii: true,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs verbose messages.
      */
-    verbose(message: string, correlationId?: string): void {
+    verbose(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Verbose,
             containsPii: false,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs verbose messages with PII.
      */
-    verbosePii(message: string, correlationId?: string): void {
+    verbosePii(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Verbose,
             containsPii: true,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs trace messages.
      */
-    trace(message: string, correlationId?: string): void {
+    trace(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Trace,
             containsPii: false,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 
     /**
      * Logs trace messages with PII.
      */
-    tracePii(message: string, correlationId?: string): void {
+    tracePii(message: string, correlationId: string): void {
         this.logMessage(message, {
             logLevel: LogLevel.Trace,
             containsPii: true,
-            correlationId: correlationId || Constants.EMPTY_STRING,
+            correlationId: correlationId,
         });
     }
 

package/src/network/ThrottlingUtils.ts

@@ -5,11 +5,7 @@
 
 import { NetworkResponse } from "./NetworkResponse.js";
 import { ServerAuthorizationTokenResponse } from "../response/ServerAuthorizationTokenResponse.js";
-import {
-    HeaderNames,
-    ThrottlingConstants,
-    Constants,
-} from "../utils/Constants.js";
+import * as Constants from "../utils/Constants.js";
 import { CacheManager } from "../cache/CacheManager.js";
 import { ServerError } from "../error/ServerError.js";
 import {
@@ -26,9 +22,7 @@ export class ThrottlingUtils {
      * @param thumbprint
      */
     static generateThrottlingStorageKey(thumbprint: RequestThumbprint): string {
-        return `${ThrottlingConstants.THROTTLING_PREFIX}.${JSON.stringify(
-            thumbprint
-        )}`;
+        return `${Constants.THROTTLING_PREFIX}.${JSON.stringify(thumbprint)}`;
     }
 
     /**
@@ -42,7 +36,7 @@ export class ThrottlingUtils {
         correlationId: string
     ): void {
         const key = ThrottlingUtils.generateThrottlingStorageKey(thumbprint);
-        const value = cacheManager.getThrottlingCache(key);
+        const value = cacheManager.getThrottlingCache(key, correlationId);
 
         if (value) {
             if (value.throttleTime < Date.now()) {
@@ -50,7 +44,7 @@ export class ThrottlingUtils {
                 return;
             }
             throw new ServerError(
-                value.errorCodes?.join(" ") || Constants.EMPTY_STRING,
+                value.errorCodes?.join(" ") || "",
                 value.errorMessage,
                 value.subError
             );
@@ -75,7 +69,9 @@ export class ThrottlingUtils {
         ) {
             const thumbprintValue: ThrottlingEntity = {
                 throttleTime: ThrottlingUtils.calculateThrottleTime(
-                    parseInt(response.headers[HeaderNames.RETRY_AFTER])
+                    parseInt(
+                        response.headers[Constants.HeaderNames.RETRY_AFTER]
+                    )
                 ),
                 error: response.body.error,
                 errorCodes: response.body.error_codes,
@@ -112,7 +108,9 @@ export class ThrottlingUtils {
     ): boolean {
         if (response.headers) {
             return (
-                response.headers.hasOwnProperty(HeaderNames.RETRY_AFTER) &&
+                response.headers.hasOwnProperty(
+                    Constants.HeaderNames.RETRY_AFTER
+                ) &&
                 (response.status < 200 || response.status >= 300)
             );
         }
@@ -130,9 +128,8 @@ export class ThrottlingUtils {
         return Math.floor(
             Math.min(
                 currentSeconds +
-                    (time || ThrottlingConstants.DEFAULT_THROTTLE_TIME_SECONDS),
-                currentSeconds +
-                    ThrottlingConstants.DEFAULT_MAX_THROTTLE_TIME_SECONDS
+                    (time || Constants.DEFAULT_THROTTLE_TIME_SECONDS),
+                currentSeconds + Constants.DEFAULT_MAX_THROTTLE_TIME_SECONDS
             ) * 1000
         );
     }

package/src/packageMetadata.ts

@@ -1,3 +1,3 @@
 /* eslint-disable header/header */
 export const name = "@azure/msal-common";
-export const version = "15.12.0";
+export const version = "16.0.0-alpha.0";

package/src/protocol/Authorize.ts

@@ -26,7 +26,6 @@ import {
     isInteractionRequiredError,
 } from "../error/InteractionRequiredAuthError.js";
 import { ServerError } from "../error/ServerError.js";
-import { StringDict } from "../utils/MsalTypes.js";
 
 /**
  * Returns map of parameters that are applicable to all calls to /authorize whether using PKCE or EAR
@@ -94,7 +93,8 @@ export function getStandardAuthorizeRequestParameters(
         if (request.sid && request.prompt === PromptValue.NONE) {
             // SessionID is only used in silent calls
             logger.verbose(
-                "createAuthCodeUrlQueryString: Prompt is none, adding sid from request"
+                "createAuthCodeUrlQueryString: Prompt is none, adding sid from request",
+                request.correlationId
             );
             RequestParameterBuilder.addSid(parameters, request.sid);
             performanceClient?.addFields(
@@ -107,7 +107,8 @@ export function getStandardAuthorizeRequestParameters(
 
             if (accountLoginHintClaim && request.domainHint) {
                 logger.warning(
-                    `AuthorizationCodeClient.createAuthCodeUrlQueryString: "domainHint" param is set, skipping opaque "login_hint" claim. Please consider not passing domainHint`
+                    `AuthorizationCodeClient.createAuthCodeUrlQueryString: "domainHint" param is set, skipping opaque "login_hint" claim. Please consider not passing domainHint`,
+                    request.correlationId
                 );
                 accountLoginHintClaim = null;
             }
@@ -115,7 +116,8 @@ export function getStandardAuthorizeRequestParameters(
             // If login_hint claim is present, use it over sid/username
             if (accountLoginHintClaim) {
                 logger.verbose(
-                    "createAuthCodeUrlQueryString: login_hint claim present on account"
+                    "createAuthCodeUrlQueryString: login_hint claim present on account",
+                    request.correlationId
                 );
                 RequestParameterBuilder.addLoginHint(
                     parameters,
@@ -132,7 +134,8 @@ export function getStandardAuthorizeRequestParameters(
                     RequestParameterBuilder.addCcsOid(parameters, clientInfo);
                 } catch (e) {
                     logger.verbose(
-                        "createAuthCodeUrlQueryString: Could not parse home account ID for CCS Header"
+                        "createAuthCodeUrlQueryString: Could not parse home account ID for CCS Header",
+                        request.correlationId
                     );
                 }
             } else if (accountSid && request.prompt === PromptValue.NONE) {
@@ -141,7 +144,8 @@ export function getStandardAuthorizeRequestParameters(
                  * SessionId is only used in silent calls
                  */
                 logger.verbose(
-                    "createAuthCodeUrlQueryString: Prompt is none, adding sid from account"
+                    "createAuthCodeUrlQueryString: Prompt is none, adding sid from account",
+                    request.correlationId
                 );
                 RequestParameterBuilder.addSid(parameters, accountSid);
                 performanceClient?.addFields(
@@ -155,12 +159,14 @@ export function getStandardAuthorizeRequestParameters(
                     RequestParameterBuilder.addCcsOid(parameters, clientInfo);
                 } catch (e) {
                     logger.verbose(
-                        "createAuthCodeUrlQueryString: Could not parse home account ID for CCS Header"
+                        "createAuthCodeUrlQueryString: Could not parse home account ID for CCS Header",
+                        request.correlationId
                     );
                 }
             } else if (request.loginHint) {
                 logger.verbose(
-                    "createAuthCodeUrlQueryString: Adding login_hint from request"
+                    "createAuthCodeUrlQueryString: Adding login_hint from request",
+                    request.correlationId
                 );
                 RequestParameterBuilder.addLoginHint(
                     parameters,
@@ -177,7 +183,8 @@ export function getStandardAuthorizeRequestParameters(
             } else if (request.account.username) {
                 // Fallback to account username if provided
                 logger.verbose(
-                    "createAuthCodeUrlQueryString: Adding login_hint from account"
+                    "createAuthCodeUrlQueryString: Adding login_hint from account",
+                    request.correlationId
                 );
                 RequestParameterBuilder.addLoginHint(
                     parameters,
@@ -194,13 +201,15 @@ export function getStandardAuthorizeRequestParameters(
                     RequestParameterBuilder.addCcsOid(parameters, clientInfo);
                 } catch (e) {
                     logger.verbose(
-                        "createAuthCodeUrlQueryString: Could not parse home account ID for CCS Header"
+                        "createAuthCodeUrlQueryString: Could not parse home account ID for CCS Header",
+                        request.correlationId
                     );
                 }
             }
         } else if (request.loginHint) {
             logger.verbose(
-                "createAuthCodeUrlQueryString: No account, adding login_hint from request"
+                "createAuthCodeUrlQueryString: No account, adding login_hint from request",
+                request.correlationId
             );
             RequestParameterBuilder.addLoginHint(parameters, request.loginHint);
             RequestParameterBuilder.addCcsUpn(parameters, request.loginHint);
@@ -211,7 +220,8 @@ export function getStandardAuthorizeRequestParameters(
         }
     } else {
         logger.verbose(
-            "createAuthCodeUrlQueryString: Prompt is select_account, ignoring account hints"
+            "createAuthCodeUrlQueryString: Prompt is select_account, ignoring account hints",
+            request.correlationId
         );
     }
 
@@ -265,15 +275,9 @@ export function getStandardAuthorizeRequestParameters(
  */
 export function getAuthorizeUrl(
     authority: Authority,
-    requestParameters: Map<string, string>,
-    encodeParams?: boolean,
-    extraQueryParameters?: StringDict | undefined
+    requestParameters: Map<string, string>
 ): string {
-    const queryString = mapToQueryString(
-        requestParameters,
-        encodeParams,
-        extraQueryParameters
-    );
+    const queryString = mapToQueryString(requestParameters);
     return UrlString.appendQueryString(
         authority.authorizationEndpoint,
         queryString

package/src/request/AuthenticationHeaderParser.ts

@@ -7,7 +7,7 @@ import {
     createClientConfigurationError,
     ClientConfigurationErrorCodes,
 } from "../error/ClientConfigurationError.js";
-import { Constants, HeaderNames } from "../utils/Constants.js";
+import { HeaderNames } from "../utils/Constants.js";
 
 type WWWAuthenticateChallenges = {
     nonce?: string;
@@ -82,9 +82,7 @@ export class AuthenticationHeaderParser {
         challenges.forEach((challenge: string) => {
             const [key, value] = challenge.split("=");
             // Remove escaped quotation marks (', ") from challenge string to keep only the challenge value
-            challengeMap[key] = unescape(
-                value.replace(/['"]+/g, Constants.EMPTY_STRING)
-            );
+            challengeMap[key] = unescape(value.replace(/['"]+/g, ""));
         });
 
         return challengeMap;