Fix syntax error in sessionStorage key for interaction status (#8346)

Fix syntax error in sessionStorage key for interaction status

Author: konstantin-msft

change/@azure-msal-browser-6498164c-5e2e-494a-b549-f9feefd7bc9e.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Fix syntax error in sessionStorage key for interaction status #8346",
+  "packageName": "@azure/msal-browser",
+  "email": "kshabelko@microsoft.com",
+  "dependentChangeType": "patch"
+}

lib/msal-browser/src/redirect_bridge/index.ts

@@ -66,7 +66,7 @@ export async function broadcastResponseToMainFrame(
         };
 
         let navigateToUrl = "";
-        const interactionKey = `${PREFIX}.${TemporaryCacheKeys.INTERACTION_STATUS_KEY}}`;
+        const interactionKey = `${PREFIX}.${TemporaryCacheKeys.INTERACTION_STATUS_KEY}`;
         try {
             /*
              * Retrieve the original navigation URL from sessionStorage

lib/msal-browser/test/redirect_bridge/broadcastResponseToMainFrame.spec.ts

@@ -211,7 +211,7 @@ describe("broadcastResponseToMainFrame", () => {
             const cachedOriginUrl = "https://localhost:8081/custom-page.html";
 
             // Set up sessionStorage with interaction status containing clientId and type
-            mockSessionStorage[`msal.interaction.status}`] = JSON.stringify({
+            mockSessionStorage[`msal.interaction.status`] = JSON.stringify({
                 clientId: testClientId,
                 type: "redirect",
             });
@@ -295,7 +295,7 @@ describe("broadcastResponseToMainFrame", () => {
             const testClientId = "hybrid-client-id";
 
             // Set up sessionStorage with interaction status containing clientId and type
-            mockSessionStorage[`msal.interaction.status}`] = JSON.stringify({
+            mockSessionStorage[`msal.interaction.status`] = JSON.stringify({
                 clientId: testClientId,
                 type: "redirect",
             });

samples/msal-browser-samples/VanillaJSTestApp2.0/app/customizable-e2e-test/test/browserAAD.spec.ts

@@ -201,6 +201,41 @@ describe("AAD-Prod Tests", () => {
             });
         });
 
+        it("Redirect bridge reads cache keys for ORIGIN_URI during redirect flow", async () => {
+            const customStartPage = sampleHomeUrl + "?redirectBridgeOriginUriTest=1";
+            const redirectStartPageRequest: RedirectRequest = {
+                ...aadTokenRequest,
+                redirectStartPage: customStartPage,
+            };
+            fs.writeFileSync(
+                "./app/customizable-e2e-test/testConfig.json",
+                JSON.stringify({
+                    msalConfig: aadMsalConfig,
+                    request: redirectStartPageRequest,
+                })
+            );
+            await page.reload();
+            await pcaInitializedPoller(page, 5000);
+
+            const testName = "redirectBridgeCacheKeys";
+            const screenshot = new Screenshot(
+                `${SCREENSHOT_BASE_FOLDER_NAME}/${testName}`
+            );
+
+            await clickLoginRedirect(screenshot, page);
+
+            await enterCredentials(page, screenshot, username, accountPwd);
+            await waitForReturnToApp(screenshot, page);
+
+            // Verify the browser navigated back to the custom redirectStartPage
+            expect(page.url()).toContain("redirectBridgeOriginUriTest=1");
+
+            // Verify browser cache contains Account, idToken, AccessToken and RefreshToken
+            await BrowserCache.verifyTokenStore({
+                scopes: aadTokenRequest.scopes,
+            });
+        });
+
         it("Performs loginPopup", async () => {
             const testName = "popupBaseCase";
             const screenshot = new Screenshot(

samples/msal-browser-samples/VanillaJSTestApp2.0/app/customizable-e2e-test/test/browserAADTenanted.spec.ts

@@ -201,6 +201,41 @@ describe("AAD-Prod Tests", () => {
             });
         });
 
+        it("Redirect bridge reads cache keys for ORIGIN_URI during redirect flow", async () => {
+            const customStartPage = sampleHomeUrl + "?redirectBridgeOriginUriTest=1";
+            const redirectStartPageRequest: RedirectRequest = {
+                ...aadTokenRequest,
+                redirectStartPage: customStartPage,
+            };
+            fs.writeFileSync(
+                "./app/customizable-e2e-test/testConfig.json",
+                JSON.stringify({
+                    msalConfig: aadMsalConfig,
+                    request: redirectStartPageRequest,
+                })
+            );
+            await page.reload();
+            await pcaInitializedPoller(page, 5000);
+
+            const testName = "redirectBridgeCacheKeys";
+            const screenshot = new Screenshot(
+                `${SCREENSHOT_BASE_FOLDER_NAME}/${testName}`
+            );
+
+            await clickLoginRedirect(screenshot, page);
+
+            await enterCredentials(page, screenshot, username, accountPwd);
+            await waitForReturnToApp(screenshot, page);
+
+            // Verify the browser navigated back to the custom redirectStartPage
+            expect(page.url()).toContain("redirectBridgeOriginUriTest=1");
+
+            // Verify browser cache contains Account, idToken, AccessToken and RefreshToken
+            await BrowserCache.verifyTokenStore({
+                scopes: aadTokenRequest.scopes,
+            });
+        });
+
         it("Performs loginPopup", async () => {
             const testName = "popupBaseCase";
             const screenshot = new Screenshot(