@azure/msal-common 16.1.0 → 16.3.0

package/src/response/ResponseHandler.ts

@@ -450,6 +450,11 @@ export class ResponseHandler {
                 userAssertionHash,
                 serverTokenResponse.key_id
             );
+            // Set resource (to be used for MCP scenarios)
+            const resource = request.resource || null;
+            if (resource) {
+                cachedAccessToken.resource = resource;
+            }
         }
 
         // refreshToken

package/src/telemetry/performance/PerformanceClient.ts

@@ -11,6 +11,7 @@ import {
     PerformanceCallbackFunction,
 } from "./IPerformanceClient.js";
 import {
+    EXT_FIELD_PREFIX,
     IntFields,
     PerformanceEvent,
     PerformanceEventContext,
@@ -453,9 +454,15 @@ export abstract class PerformanceClient implements IPerformanceClient {
             addError(error, this.logger, rootEvent);
         }
 
-        // Add sub-measurement attribute to root event.
+        // Add sub-measurement attribute to root event's ext field.
         if (!isRoot) {
-            rootEvent[event.name + "DurationMs"] = Math.floor(event.durationMs);
+            rootEvent.ext = {
+                ...rootEvent.ext,
+                ...event.ext,
+            };
+            rootEvent.ext[event.name + "DurationMs"] = Math.floor(
+                event.durationMs
+            );
             return { ...rootEvent };
         }
 
@@ -521,10 +528,36 @@ export abstract class PerformanceClient implements IPerformanceClient {
     ): void {
         const event = this.eventsByCorrelationId.get(correlationId);
         if (event) {
-            this.eventsByCorrelationId.set(correlationId, {
+            const staticFields: { [key: string]: {} | undefined } = {};
+            const dynamicFields: Record<string, string | number> = {};
+
+            for (const key in fields) {
+                if (key.startsWith(EXT_FIELD_PREFIX)) {
+                    const dynamicKey = key.slice(EXT_FIELD_PREFIX.length);
+                    const value = fields[key];
+                    if (
+                        typeof value === "string" ||
+                        typeof value === "number"
+                    ) {
+                        dynamicFields[dynamicKey] = value;
+                    }
+                } else {
+                    staticFields[key] = fields[key];
+                }
+            }
+
+            const updatedEvent: PerformanceEvent = {
                 ...event,
-                ...fields,
-            });
+                ...staticFields,
+            };
+            if (Object.keys(dynamicFields).length) {
+                updatedEvent.ext = {
+                    ...updatedEvent.ext,
+                    ...dynamicFields,
+                };
+            }
+
+            this.eventsByCorrelationId.set(correlationId, updatedEvent);
         } else {
             this.logger.trace(
                 "PerformanceClient: Event not found for",
@@ -545,12 +578,29 @@ export abstract class PerformanceClient implements IPerformanceClient {
         const event = this.eventsByCorrelationId.get(correlationId);
         if (event) {
             for (const counter in fields) {
-                if (!event.hasOwnProperty(counter)) {
-                    event[counter] = 0;
-                } else if (isNaN(Number(event[counter]))) {
-                    return;
+                if (counter.startsWith(EXT_FIELD_PREFIX)) {
+                    event.ext = event.ext || {};
+                    // Route to ext sub-object
+                    const dynamicKey = counter.slice(EXT_FIELD_PREFIX.length);
+                    const currentValue = event.ext[dynamicKey];
+                    if (currentValue === undefined) {
+                        event.ext[dynamicKey] = 0;
+                    } else if (isNaN(Number(currentValue))) {
+                        return;
+                    }
+                    event.ext[dynamicKey] =
+                        (Number(event.ext[dynamicKey]) || 0) +
+                        (fields[counter] ?? 0);
+                } else {
+                    /* eslint-disable custom-msal/no-dynamic-telemetry-fields -- internal dispatching of static fields by name */
+                    if (!event.hasOwnProperty(counter)) {
+                        event[counter] = 0;
+                    } else if (isNaN(Number(event[counter]))) {
+                        return;
+                    }
+                    event[counter] += fields[counter];
+                    /* eslint-enable custom-msal/no-dynamic-telemetry-fields */
                 }
-                event[counter] += fields[counter];
             }
         } else {
             this.logger.trace(
@@ -674,9 +724,11 @@ export abstract class PerformanceClient implements IPerformanceClient {
      */
     private truncateIntegralFields(event: PerformanceEvent): void {
         this.intFields.forEach((key) => {
+            /* eslint-disable custom-msal/no-dynamic-telemetry-fields -- internal truncation of known integer fields */
             if (key in event && typeof event[key] === "number") {
                 event[key] = Math.floor(event[key]);
             }
+            /* eslint-enable custom-msal/no-dynamic-telemetry-fields */
         });
     }
 

package/src/telemetry/performance/PerformanceEvent.ts

@@ -191,10 +191,23 @@ export type PerformanceEvent = {
     visibilityChangeCount?: number;
     incompleteSubsCount?: number;
 
+    /**
+     * Network connection info from the Network Information API (Chromium only).
+     * Read from: https://developer.mozilla.org/docs/Web/API/NetworkInformation
+     */
+    networkEffectiveType?: string;
+    networkRtt?: number;
+
     /**
      * CorrelationId of the in progress iframe request that was awaited
      */
     awaitIframeCorrelationId?: string;
+    /**
+     * Monitor_window_timeout debugging telemetry
+     */
+    redirectBridgeTimeoutMs?: number;
+    isRedirectUriCrossOrigin?: boolean;
+    redirectBridgeMessageVersion?: number;
 
     /**
      * Size of the id token
@@ -321,6 +334,8 @@ export type PerformanceEvent = {
     extRtExpiresOnSeconds?: number;
     rtOffsetSeconds?: number;
 
+    sidFromClaim?: boolean;
+    // Backward-compatible alias for sidFromClaim
     sidFromClaims?: boolean;
     sidFromRequest?: boolean;
     loginHintFromRequest?: boolean;
@@ -344,6 +359,34 @@ export type PerformanceEvent = {
     // Hashed logs in the format [millis1,hash1;millis2,hash2;...]
     logs?: string;
 
+    // Whether the application is configured for MCP flows
+    isMcp?: boolean;
+
+    /**
+     * Source of cloud discovery metadata (config, cache, network, hardcoded_values)
+     */
+    cloudDiscoverySource?: string;
+
+    /**
+     * Source of authority endpoint metadata (config, cache, network, hardcoded_values)
+     */
+    authorityEndpointSource?: string;
+
+    /**
+     * Number of accounts removed during cache cleanup
+     */
+    accountsRemoved?: number;
+
+    /**
+     * Number of access tokens removed during cache cleanup
+     */
+    accessTokensRemoved?: number;
+
+    /**
+     * Number of failures when removing token binding keys
+     */
+    removeTokenBindingKeyFailure?: number;
+
     /**
      * Reason for silent refresh fallback to iframe
      * Format: errorCode or errorCode|subError
@@ -351,6 +394,82 @@ export type PerformanceEvent = {
      * @type {?string}
      */
     silentRefreshReason?: string;
+
+    /**
+     * Whether this request was deduped with another in-flight request
+     */
+    deduped?: boolean;
+
+    /**
+     * Whether the user has "Keep Me Signed In" enabled
+     */
+    kmsi?: boolean;
+
+    /**
+     * Whether this event was executed in the background
+     */
+    isBackground?: boolean;
+
+    /**
+     * Cache migration telemetry — pre-migration counts
+     */
+    preMigrateAcntCount?: number;
+    preMigrateATCount?: number;
+    preMigrateITCount?: number;
+    preMigrateRTCount?: number;
+
+    /**
+     * Cache migration telemetry — post-migration counts
+     */
+    postMigrateAcntCount?: number;
+    postMigrateATCount?: number;
+    postMigrateITCount?: number;
+    postMigrateRTCount?: number;
+
+    /**
+     * Cache migration telemetry — old schema counts
+     */
+    oldAcntCount?: number;
+    oldATCount?: number;
+    oldITCount?: number;
+    oldRTCount?: number;
+
+    /**
+     * Cache migration telemetry — skipped and migrated counts
+     */
+    skipATMigrateCount?: number;
+    skipITMigrateCount?: number;
+    skipRTMigrateCount?: number;
+    migratedATCount?: number;
+    migratedITCount?: number;
+    migratedRTCount?: number;
+
+    /**
+     * Cache telemetry — expired, invalid, and removed counts
+     */
+    expiredCacheRemovedCount?: number;
+    expiredAcntRemovedCount?: number;
+    invalidCacheCount?: number;
+
+    /**
+     * Encrypted cache telemetry
+     */
+    unencryptedCacheCount?: number;
+    encryptedCacheCount?: number;
+    encryptedCacheExpiredCount?: number;
+    encryptedCacheCorruptionCount?: number;
+
+    /**
+     * Container for dynamically-named telemetry fields.
+     * Fields whose names are constructed at runtime (e.g., "[eventName]CallCount")
+     * should be stored here instead of being set as top-level properties.
+     * Use the "ext." prefix when calling addFields/incrementFields to automatically
+     * route fields to this sub-object.
+     *
+     * @remarks
+     * This property is typed as `Record<string, string | number>`.
+     */
+    ext?: Record<string, string | number>;
 };
 
 export type PerformanceEventContext = {
@@ -365,6 +484,13 @@ export type PerformanceEventStackedContext = PerformanceEventContext & {
     childErr?: string;
 };
 
+/**
+ * Prefix used to mark telemetry field names as dynamic.
+ * Fields with this prefix in addFields/incrementFields calls will be routed
+ * to the PerformanceEvent.ext sub-object.
+ */
+export const EXT_FIELD_PREFIX = "ext.";
+
 export const IntFields: ReadonlySet<string> = new Set([
     "accessTokenSize",
     "durationMs",
@@ -389,4 +515,7 @@ export const IntFields: ReadonlySet<string> = new Set([
     "currRefreshCount",
     "expiredCacheRemovedCount",
     "upgradedCacheCount",
+    "networkRtt",
+    "redirectBridgeTimeoutMs",
+    "redirectBridgeMessageVersion",
 ]);

package/src/utils/FunctionWrappers.ts

@@ -33,8 +33,10 @@ export const invoke = <T extends Array<any>, U>(
         );
         if (correlationId) {
             // Track number of times this API is called in a single request
-            const eventCount = eventName + "CallCount";
-            telemetryClient.incrementFields({ [eventCount]: 1 }, correlationId);
+            telemetryClient.incrementFields(
+                { [`ext.${eventName}CallCount`]: 1 },
+                correlationId
+            );
         }
         try {
             const result = callback(...args);
@@ -89,8 +91,10 @@ export const invokeAsync = <T extends Array<any>, U>(
         );
         if (correlationId) {
             // Track number of times this API is called in a single request
-            const eventCount = eventName + "CallCount";
-            telemetryClient.incrementFields({ [eventCount]: 1 }, correlationId);
+            telemetryClient.incrementFields(
+                { [`ext.${eventName}CallCount`]: 1 },
+                correlationId
+            );
         }
         return callback(...args)
             .then((response) => {