Worker runtime: avoid errors in JSON.parse to bubble up

duckdb · Feb 29, 2024 · 98daef7 · 98daef7
1 parent f11c8df
commit 98daef7
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/packages/duckdb-wasm/src/bindings/runtime_browser.ts b/packages/duckdb-wasm/src/bindings/runtime_browser.ts
@@ -44,13 +44,18 @@ export const BROWSER_RUNTIME: DuckDBRuntime & {
             }
             const infoStr = readString(mod, d, n);
             dropResponseBuffers(mod);
-            const info = JSON.parse(infoStr);
-            if (info == null) {
+            try {
+                const info = JSON.parse(infoStr);
+                if (info == null) {
+                   return null;
+                }
+                const file = { ...info, blob: null } as DuckDBFileInfo;
+                BROWSER_RUNTIME._fileInfoCache.set(fileId, file);
+                return file;
+            } catch (error) {
+		console.warn(error);
                 return null;
             }
-            const file = { ...info, blob: null } as DuckDBFileInfo;
-            BROWSER_RUNTIME._fileInfoCache.set(fileId, file);
-            return file;
         } catch (e: any) {
             console.log(e);
             return null;