post_device_status discards all control errors and always returns 204

Abierto
#33 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
55/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
rust
Área
api, backend

Línea de trabajo

Read crates/api/src/handlers.rs:98-113 and compare the control error handling with lib.rs:204-208. Trace how handler errors become HTTP responses, then check how PoddControl::send reports a closed command channel. Done when failed control sends are logged or return a 5xx instead of 204; no test file is named in the issue.

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

Descripción

bug

Every control call in post_device_status is fired with let _ = … (crates/api/src/handlers.rs:98-113): set_power, set_target_temp, clear_alarm, prime, apply_device_settings. PoddControl::send returns Err only when the command mpsc is closed — i.e. the control core has died (exactly the failure mode lib.rs:204-208 was hardened against). In that state the handler still returns 204 No Content, so the UI believes a temperature/alarm change was applied when nothing received it. At minimum log these errors; ideally surface a 5xx when the command channel is gone.


Filed from a repo-wide code audit (working tree at bcad746).

Lenguaje dominante
Rust
Estrellas
0
Forks
0
Métricas de merge de PR
Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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 GoogleBot42/podd

Todos los issues de GoogleBot42/podd

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.