mirror/ditto
mirror/ditto
1
0
Fork 0
mirror of https://gitlab.com/soapbox-pub/ditto.git synced 2025-12-06 03:19:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge branch 'req-abort' into 'main'

Browse source 
Fix REQ abort logic

See merge request soapbox-pub/ditto!712
This commit is contained in:
Alex Gleason 2025-03-08 04:59:24 +00:00
commit d359e832ed
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
packages/ditto/controllers/nostr/relay.ts
View file

@ -101,6 +101,10 @@ function connectStream(socket: WebSocket, ip: string | undefined, opts: ConnectS
// HACK: Due to a bug in Deno, we need to call the close handler manually.
// https://github.com/denoland/deno/issues/27924
function closeSocket(code?: number, reason?: string): void {
for (const controller of controllers.values()) {
controller.abort();
}
send(['NOTICE', `closed: ${reason} (${code})`]);
socket.close(code, reason);
handleSocketClose();
}
@ -152,9 +156,10 @@ function connectStream(socket: WebSocket, ip: string | undefined, opts: ConnectS
const controller = new AbortController();
controllers.get(subId)?.abort();
controllers.set(subId, controller);
const signal = controller.signal;
try {
for await (const msg of relay.req(filters, { limit: 100, timeout: conf.db.timeouts.relay })) {
for await (const msg of relay.req(filters, { limit: 100, signal, timeout: conf.db.timeouts.relay })) {
if (msg[0] === 'EVENT') {
const [, , event] = msg;
send(['EVENT', subId, purifyEvent(event)]);
@ -171,8 +176,8 @@ function connectStream(socket: WebSocket, ip: string | undefined, opts: ConnectS
} else {
send(['CLOSED', subId, 'error: something went wrong']);
}
} finally {
controllers.delete(subId);
return;
}
}