Dashboard: a successful response whose JSON does not parse is treated as an empty success

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

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

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

Línea de trabajo

Empieza leyendo frontend/src/services/api.ts alrededor de readSuccessBody y la rama de error no JSON inmediatamente anterior. Confirma que las respuestas 204/205 vacías se gestionan por separado y, a continuación, cambia la ruta de JSON malformado para que genere el mismo tipo de ApiError específico de la ruta. Verifícalo con las pruebas relevantes del servicio frontend, incluida una respuesta JSON 2xx con un body truncado o no válido.

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

Descripción

bug post-launch

readSuccessBody in frontend/src/services/api.ts (lines ~370-375) returns undefined when a 2xx response with a JSON content type has a body that does not parse:

try {
  return (await response.json()) as T;
} catch {
  return undefined as T;
}

So a truncated or malformed successful body is treated as a successful empty result. The caller then shows success, or fails later on a property of undefined with a message that does not say what went wrong. Empty 204/205 responses are already handled before this point and are not affected.

Expected: a body that the content type says is JSON but that does not parse raises an ApiError naming the path (as the non-JSON branch above already does), so the page shows a clear error. Found in a code review (2026-10-06); no user report.

Lenguaje dominante
Python
Estrellas
0
Forks
4
Merge medio
6 h 5 min
PR fusionados (30 d)
453

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 Controlvar/controlvar

Todos los issues de Controlvar/controlvar

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.