world-vercel waitForTerminalStatus drops the abort signal on its fallback read
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 76/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend
Línea de trabajo
Empieza en waitForWorkflowRunTerminalStatus y sigue las llamadas de respaldo a getWorkflowRun. Después, sigue hasta readRun para ver cómo se pasan las opciones de solicitud. Comprueba las lecturas de respaldo después de una respuesta con un estado no compatible, durante la ventana de estado no compatible y cuando el presupuesto es cero; el trabajo está hecho cuando una espera cancelada se rechaza de inmediato en cada ruta. Ejecuta las pruebas pertinentes para la espera de un estado terminal y añade cobertura para el comportamiento de cancelación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
waitForWorkflowRunTerminalStatus passes params.signal to the long-poll read of /v2/runs/:id/status, but its three fallback paths call getWorkflowRun(id, { resolveData }, config) without it: after the status route answers 404/405/501/504, during the five-minute unsupported window that follows, and when the budget clamps to zero. An aborted wait on those paths keeps its request open until the request timeout (60s by default) and keeps holding a connection from the default pool.
Versions: @workflow/world-vercel 5.0.1, same code on main at 2d2159e.
// node repro.mjs fallback|native
import http from 'node:http';
const mode = process.argv[2];
const server = http.createServer((req, res) => {
if (req.url.includes('/status') && mode === 'fallback') res.writeHead(404).end();
// otherwise never answer
});
await new Promise((r) => server.listen(0, '127.0.0.1', r));
process.env.VERCEL_WORKFLOW_SERVER_URL = `http://127.0.0.1:${server.address().port}`;
const { createStorage } = await import('@workflow/world-vercel');
const runs = createStorage({ token: 'fake-token' }).runs;
const t = Date.now();
const out = await runs
.waitForTerminalStatus('wrun_01JZZZZZZZZZZZZZZZZZZZZZZZ', { timeoutMs: 30_000, signal: AbortSignal.timeout(1_000) })
.then(() => 'resolved', (e) => `rejected ${e.name}`);
console.log(mode, out, `${Date.now() - t}ms`);
process.exit(0);
native rejected WorkflowWorldError 1007ms
fallback rejected WorkflowWorldError 60013ms
Expected: the wait rejects about 1s after the signal aborts on every path.
Actual: on the fallback read it rejects only when the 60s request timeout fires.
Possible fix: forward signal through getWorkflowRun to readRun on the fallback calls, and check signal.throwIfAborted() at entry.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.5k
- Forks
- 385
- Merge medio
- 22 h 51 min
- PR fusionados (30 d)
- 171
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de vercel/workflow
-
node-module-esbuild-plugin: getImportedIdentifier only checks the first named import when locating a violationPosiblemente ocupada @FelixFangX la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
vercel/workflow#5057 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Feature: Expose isLastAttempt/maxAttempts in step metadataPosiblemente ocupada @joelfickson la tomó hace 2 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
vercel/workflow#4979 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
WorkflowServerReadableStream leaks the connection when cancelled before streams.get() resolvesPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
vercel/workflow#4644 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
world-local write() does not retry a transient Windows EPERM from its existence probePosiblemente ocupada @pranaygp la tomó hace 33 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
vercel/workflow#4203 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de vercel/workflow
Issues similares
-
agent-reported area/react bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Nimblesite/SharpLsp#510 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
backend refactor
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
NIAEFEUP/tts#927 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
callstackincubator/react-native-harness#296 ·
Los mantenedores suelen responder en 1 día