Fix unhandled exception in NAA JS Runtime scenario (AzureAD#7475)

There were regressions to the NAA JS Runtime scenario by the following commits, adding new platform API dependencies in the NAA critical path. This change is tested to get JS Runtime back into a working state. - AzureAD@df26209 - AzureAD@09066cc --------- Co-authored-by: Hector Morales <hemoral@microsoft.com>

Author: codexeon

change/@azure-msal-browser-abd084f2-52c9-4078-b44c-5680a1e2f431.json

@@ -0,0 +1,7 @@
+{
+ "type": "patch",
+ "comment": "Fix unhandled exception in NAA JS Runtime scenario (#7475)",
+ "packageName": "@azure/msal-browser",
+ "email": "dasau@microsoft.com",
+ "dependentChangeType": "patch"
+}

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

@@ -102,7 +102,7 @@ export class NestedAppAuthController implements IController {
 
  // Initialize the crypto class.
  this.browserCrypto = operatingContext.isBrowserEnvironment()
- ? new CryptoOps(this.logger, this.performanceClient)
+ ? new CryptoOps(this.logger, this.performanceClient, true)
  : DEFAULT_CRYPTO_IMPLEMENTATION;
 
  // Initialize the browser storage class.

lib/msal-browser/src/crypto/BrowserCrypto.ts

@@ -47,7 +47,9 @@ const keygenAlgorithmOptions: RsaHashedKeyGenParams = {
 /**
  * Check whether browser crypto is available.
  */
-export function validateCryptoAvailable(): void {
+export function validateCryptoAvailable(
+ skipValidateSubtleCrypto: boolean
+): void {
  if (!window) {
  throw createBrowserAuthError(
  BrowserAuthErrorCodes.nonBrowserEnvironment
@@ -56,7 +58,7 @@ export function validateCryptoAvailable(): void {
  if (!window.crypto) {
  throw createBrowserAuthError(BrowserAuthErrorCodes.cryptoNonExistent);
  }
- if (!window.crypto.subtle) {
+ if (!skipValidateSubtleCrypto && !window.crypto.subtle) {
  throw createBrowserAuthError(
  BrowserAuthErrorCodes.cryptoNonExistent,
  SUBTLE_SUBERROR

lib/msal-browser/src/crypto/CryptoOps.ts

@@ -50,10 +50,16 @@ export class CryptoOps implements ICrypto {
  private static EXTRACTABLE: boolean = true;
  private cache: AsyncMemoryStorage<CachedKeyPair>;
 
- constructor(logger: Logger, performanceClient?: IPerformanceClient) {
+ constructor(
+ logger: Logger,
+ performanceClient?: IPerformanceClient,
+ skipValidateSubtleCrypto?: boolean
+ ) {
  this.logger = logger;
  // Browser crypto needs to be validated first before any other classes can be set.
- BrowserCrypto.validateCryptoAvailable();
+ BrowserCrypto.validateCryptoAvailable(
+ skipValidateSubtleCrypto ?? false
+ );
  this.cache = new AsyncMemoryStorage<CachedKeyPair>(this.logger);
  this.performanceClient = performanceClient;
  }

lib/msal-browser/src/utils/BrowserUtils.ts

@@ -67,7 +67,9 @@ export function isInPopup(): boolean {
  * Returns current window URL as redirect uri
  */
 export function getCurrentUri(): string {
- return window.location.href.split("?")[0].split("#")[0];
+ return typeof window !== "undefined" && window.location
+ ? window.location.href.split("?")[0].split("#")[0]
+ : "";
 }
 
 /**