fix: use uuid package instead of crypto for remote http jupyters (#245)

uwdata · Jan 21, 2024 · 12523a0 · 12523a0
1 parent 4114465
commit 12523a0
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/widget/package.json b/packages/widget/package.json
@@ -17,7 +17,8 @@
   },
   "dependencies": {
     "@uwdata/vgplot": "^0.4.0",
-    "apache-arrow": "^13.0.0"
+    "apache-arrow": "^13.0.0",
+    "uuid": "^9.0.1"
   },
   "devDependencies": {
     "anywidget": "^0.8.0"

diff --git a/packages/widget/src/index.js b/packages/widget/src/index.js
@@ -3,6 +3,7 @@
 import { coordinator, namedPlots, parseSpec } from '@uwdata/vgplot';
 import * as arrow from 'apache-arrow';
 import './style.css';
+import { v4 as uuidv4 } from 'uuid';
 
 /**
  * @typedef Model
@@ -30,7 +31,7 @@ export async function render(view) {
    * @param {(reason?: any) => void} reject the promise reject callback
    */
   function send(query, resolve, reject) {
-    const uuid = globalThis.crypto.randomUUID();
+    const uuid = uuidv4();
 
     openQueries.set(uuid, { query, startTime: performance.now(), resolve, reject });
     view.model.send({ ...query, uuid });