partysocket: a URL provider that rejects with undefined gives an unhandled rejection and stops the reconnection

Abierto Apto para principiantes
  • ¿Algo no está bien?
#434 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
62/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
typescript
Área
networking

Línea de trabajo

El error está en el .catch() de _connect() en packages/partysocket/src/ws.ts (alrededor de las líneas 585-588), que lee err.message de un proveedor de URL o de protocols que fue rechazado. Ejecuta el repro del issue contra partysocket 1.3.0 para confirmar el TypeError y el estado CONNECTING bloqueado. Está terminado cuando un proveedor que rechaza con undefined o null emite un evento error y programa el siguiente intento, sin unhandled rejection, y la suite de tests de partysocket pasa.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

The .catch() of _connect() reads err.message. When the URL provider (or the protocols provider) rejects with undefined or null, for example with reject() and no argument, that line throws a TypeError inside the .catch(). The result is an unhandled rejection, no error event, and no next attempt: the socket stays in CONNECTING.

https://github.com/cloudflare/partykit/blob/partysocket%401.3.0/packages/partysocket/src/ws.ts#L585-L588

Repro

The packages: [email protected]. Install them with npm i [email protected].

// npm install, then: node repro.ts (Node.js 22.18 or later runs TypeScript directly)
import WebSocket from "partysocket/ws";

process.on("unhandledRejection", (reason) => {
  console.log("unhandledRejection:", String(reason));
});

let calls = 0;
const ws = new WebSocket(
  () => {
    calls++;
    // for example, reject() in a token fetcher
    return Promise.reject(undefined);
  },
  undefined,
  { minReconnectionDelay: 50, maxReconnectionDelay: 50 },
);
let errors = 0;
ws.addEventListener("error", () => errors++);

setTimeout(() => {
  console.log("provider calls in 500 ms:", calls);
  console.log("error events:", errors);
  console.log("readyState:", ws.readyState);
  ws.close();
}, 500);

node repro.ts (Node.js 22.18 or later runs TypeScript directly)

Actual

unhandledRejection: TypeError: Cannot read properties of undefined (reading 'message')
provider calls in 500 ms: 1
error events: 0
readyState: 0

Expected

An error event, and a new attempt after the reconnection delay, as for a provider that rejects with an Error. No unhandled rejection.

A fix

A fix with a new test is on the branch fix/partysocket-provider-rejects-nullish of my fork, at 54a3d36. On a GitHub Actions runner, the new test fails before the change and passes after it, and the whole partysocket suite passes. This repository accepts pull requests only from collaborators, so I did not open a pull request. I can open one if you allow it, or you can take the commit.

I found this with AI assistance (Claude) during a review of WebSocket client libraries. The repro above was run with partysocket 1.3.0 on Node.js 24.22.2 on Linux (a GitHub Actions runner), and the output above is from that run. The fix was also prepared with AI assistance (Claude).

Lenguaje dominante
TypeScript
Estrellas
1.3k
Forks
86
Sin PR fusionados en 30 d

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de cloudflare/partykit

Todos los issues de cloudflare/partykit

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.