Bugfixes

* `crypto` didn't work in Firefox due to it being a getter
* Fix forwarding to work with message ports

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@exact-realty/lot",
-	"version": "0.0.23",
+	"version": "0.0.24",
 	"description": "Sandbox for isolating ECMAScript code",
 	"main": "./dist/index.cjs",
 	"types": "./dist/index.d.cts",

src/untrusted/impl/worker/workerSandboxManager.ts

@@ -158,35 +158,14 @@ const workerSandboxManager = async (
 		const revokeWorkerMessageEventListener = createMessageEventListener(
 			worker,
 			(data: unknown[]) => {
-				if (
-					![
-						EMessageTypes.REQUEST,
-						EMessageTypes.RESULT,
-						EMessageTypes.ERROR,
-					].includes(data[0] as EMessageTypes)
-				)
-					return;
+				if (EMessageTypes.REQUEST !== data[0]) return;
 
-				if (data[0] === EMessageTypes.REQUEST) {
-					Logger.debug(
-						'Forwarding REQUEST from worker to parent for executing task [' +
-							data[1] +
-							'] ' +
-							data[2],
-					);
-				} else {
-					Logger.debug(
-						'Forwarding ' +
-							(data[0] === EMessageTypes.RESULT
-								? 'RESULT'
-								: 'ERROR') +
-							' from worker to parent from executing task [' +
-							data[1] +
-							']',
-					);
-				}
+				Logger.debug(
+					'Forwarding REQUEST from worker to parent for executing task' +
+						data[2],
+				);
 
-				postMessageOutgoing(data);
+				postMessageOutgoing(data, [data[1] as unknown as MessagePort]);
 			},
 		);
 

src/untrusted/lib/createContext.ts

@@ -22,6 +22,7 @@ import {
 	aMap,
 	aPush,
 	fnApply,
+	fnCall,
 	oCreate,
 	oDefineProperties,
 	oDefineProperty,
@@ -166,7 +167,7 @@ const descriptorToFunctionProxy = (ctx: object, a?: PropertyDescriptor) => {
 	if (typeof a['value'] === 'function') {
 		a['value'] = createGlobalFunctionProxy(ctx, a['value']);
 	} else if (typeof a['get'] === 'function') {
-		const v = a['get'].call($global);
+		const v = fnCall(a['get'], $global);
 		if (typeof v === 'function') {
 			const nameDescriptor = oGetOwnPropertyDescriptor(a['get'], 'name');
 			const newGetter = (() => createGlobalFunctionProxy(ctx, v)).bind(
@@ -176,6 +177,11 @@ const descriptorToFunctionProxy = (ctx: object, a?: PropertyDescriptor) => {
 				oDefineProperty(newGetter, 'name', nameDescriptor);
 			}
 			a['get'] = newGetter;
+		} else {
+			a['get'] = (() => v).bind(null);
+		}
+		if (typeof a['set'] === 'function') {
+			a['set'] = (() => {}).bind(null);
 		}
 	}
 	return a;