Stop setting isBroker in initialize

change/@azure-msal-browser-08e8affc-ae18-41c2-821d-112c37fbdbdb.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Remove isBroker flag from initialize",
+  "packageName": "@azure/msal-browser",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

lib/msal-browser/apiReview/msal-browser.api.md

@@ -744,7 +744,7 @@ export interface IController {
     // (undocumented)
     hydrateCache(result: AuthenticationResult, request: SilentRequest | SsoSilentRequest | RedirectRequest | PopupRequest): Promise<void>;
     // (undocumented)
-    initialize(request?: InitializeApplicationRequest, isBroker?: boolean): Promise<void>;
+    initialize(request?: InitializeApplicationRequest): Promise<void>;
     // (undocumented)
     initializeWrapperLibrary(sku: WrapperSKU, version: string): void;
     // @internal (undocumented)

lib/msal-browser/src/app/PublicClientApplication.ts

@@ -96,7 +96,7 @@ export class PublicClientApplication implements IPublicClientApplication {
      * @param request {?InitializeApplicationRequest}
      */
     async initialize(request?: InitializeApplicationRequest): Promise<void> {
-        return this.controller.initialize(request, this.isBroker);
+        return this.controller.initialize(request);
     }
 
     /**

lib/msal-browser/src/controllers/IController.ts

@@ -29,8 +29,7 @@ import { EventType } from "../event/EventType.js";
 export interface IController {
     // TODO: Make request mandatory in the next major version?
     initialize(
-        request?: InitializeApplicationRequest,
-        isBroker?: boolean
+        request?: InitializeApplicationRequest
     ): Promise<void>;
 
     acquireTokenPopup(request: PopupRequest): Promise<AuthenticationResult>;

lib/msal-browser/src/controllers/NestedAppAuthController.ts

@@ -158,9 +158,7 @@ export class NestedAppAuthController implements IController {
      * @returns
      */
     async initialize(
-        request?: InitializeApplicationRequest,
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
-        isBroker?: boolean
+        request?: InitializeApplicationRequest
     ): Promise<void> {
         const initCorrelationId = request?.correlationId || createNewGuid();
         await this.browserStorage.initialize(initCorrelationId);

lib/msal-browser/src/controllers/StandardController.ts

@@ -287,8 +287,7 @@ export class StandardController implements IController {
      * @param request {?InitializeApplicationRequest} correlation id
      */
     async initialize(
-        request?: InitializeApplicationRequest,
-        isBroker?: boolean
+        request?: InitializeApplicationRequest
     ): Promise<void> {
         const correlationId = this.getRequestCorrelationId(request);
         this.logger.trace("initialize called", correlationId);
@@ -319,12 +318,7 @@ export class StandardController implements IController {
         );
         this.eventHandler.emitEvent(EventType.INITIALIZE_START);
 
-        // Broker applications are initialized twice, so we avoid double-counting it
-        if (!isBroker) {
-            try {
-                this.logMultipleInstances(initMeasurement, initCorrelationId);
-            } catch {}
-        }
+        this.logMultipleInstances(initMeasurement, initCorrelationId);
 
         await invokeAsync(
             this.browserStorage.initialize.bind(this.browserStorage),

lib/msal-browser/test/app/PublicClientApplication.spec.ts

@@ -682,26 +682,6 @@ describe("PublicClientApplication.ts Class Unit Tests", () => {
 
             expect(preGenerateSpy).toHaveBeenCalledTimes(1);
         });
-
-        it("passes in isBroker in request", async () => {
-            pca = new PublicClientApplication({
-                auth: {
-                    clientId: TEST_CONFIG.MSAL_CLIENT_ID,
-                },
-                system: {
-                    allowPlatformBroker: false,
-                },
-            });
-            const initializeControllerSpy = jest.spyOn(
-                StandardController.prototype,
-                "initialize"
-            );
-            await pca.initialize();
-            expect(initializeControllerSpy).toHaveBeenCalledWith(
-                undefined,
-                false
-            );
-        });
     });
 
     describe("handleRedirectPromise", () => {