fix: replace crypto.randomUUID to allow insecure environments (fix #9… (#9339)

Author: plusgut

packages/browser/src/client/public/esm-client-injector.js

@@ -9,7 +9,7 @@
     }
 
     const { evaluatedModules } = __vitest_worker__
-    const moduleId = crypto.randomUUID()
+    const moduleId = `${Math.random()}`
     const viteModule = evaluatedModules.ensureModule(moduleId, moduleId)
 
     viteModule.evaluated = false

test/browser/fixtures/insecure-context/src/basic.test.ts

@@ -0,0 +1,6 @@
+import { expect, test } from "vitest";
+
+test("basic", async () => {
+  expect(window.isSecureContext).toBe(false);
+  expect(1).toBe((await import("./dynamic-import")).default);
+});

test/browser/fixtures/insecure-context/src/dynamic-import.ts

@@ -0,0 +1 @@
+export default 1;

test/browser/fixtures/insecure-context/vitest.config.ts

@@ -0,0 +1,19 @@
+import { fileURLToPath } from "node:url";
+import os from "node:os";
+import { defineConfig } from "vitest/config";
+import { instances, provider } from "../../settings";
+
+export default defineConfig({
+  cacheDir: fileURLToPath(new URL("./node_modules/.vite", import.meta.url)),
+  test: {
+    browser: {
+      enabled: true,
+      headless: true,
+      provider,
+      instances,
+    },
+  },
+  server: {
+    host: os.hostname(), // To force an insecure-context, a host which is not 127.0.0.1 or localhost is needed
+  },
+});

test/browser/specs/insecure-context.test.ts

@@ -0,0 +1,13 @@
+import os from 'node:os'
+import { expect, test } from 'vitest'
+import { instances, runBrowserTests } from './utils'
+
+// server.host = os.hostname // doesnt work on mac, therefore the test is only run on linux
+test.runIf(os.platform() === 'linux')('server-host check dynamic import at insecure context', async () => {
+  const { stdout, stderr } = await runBrowserTests({
+    root: './fixtures/insecure-context',
+  })
+
+  expect(stderr).toBe('')
+  expect(stdout).toReportSummaryTestFiles({ passed: instances.length })
+})