Merge branch 'worker-names' into 'main'

Add names to web workers

See merge request soapbox-pub/ditto!540
This commit is contained in:
Alex Gleason 2024-10-09 22:59:15 +00:00
commit 878535c428
3 changed files with 3 additions and 2 deletions

View file

@ -5,7 +5,7 @@ import './handlers/abortsignal.ts';
import { fetchResponsesCounter } from '@/metrics.ts';
const worker = new Worker(new URL('./fetch.worker.ts', import.meta.url), { type: 'module' });
const worker = new Worker(new URL('./fetch.worker.ts', import.meta.url), { type: 'module', name: 'fetchWorker' });
const client = Comlink.wrap<typeof FetchWorker>(worker);
// Wait for the worker to be ready before we start using it.

View file

@ -20,6 +20,7 @@ class PolicyWorker implements NPolicy {
new URL('./policy.worker.ts', import.meta.url),
{
type: 'module',
name: 'PolicyWorker',
// FIXME: Disabled until Deno 2.0 adds support for `import` permission here.
// https://github.com/denoland/deno/issues/26074
// deno: {

View file

@ -4,7 +4,7 @@ import * as Comlink from 'comlink';
import type { VerifyWorker } from './verify.worker.ts';
const worker = Comlink.wrap<typeof VerifyWorker>(
new Worker(new URL('./verify.worker.ts', import.meta.url), { type: 'module' }),
new Worker(new URL('./verify.worker.ts', import.meta.url), { type: 'module', name: 'verifyEventWorker' }),
);
function verifyEventWorker(event: NostrEvent): Promise<boolean> {