Reboot / Execute / SetSettings commands report success to the client but are silently dropped
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Start in crates/podd-core/src/lib.rs:105-113 and trace how the dispatcher handles SetSettingsCbor, Reboot, Update, and Execute. Compare those paths with the API handlers in crates/api/src/handlers.rs:190-212 and control.rs:256-263 to understand how callers receive success. Decide what status callers should receive until these commands are implemented, then verify the affected API behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The API happily accepts these and returns success — post_jobs returns 204 for Job::Reboot (crates/api/src/handlers.rs:210-212), post_execute returns {success:true, message:"queued …"} (handlers.rs:190-195 via control.rs:256-263), and post_device_status forwards settings to apply_device_settings. But the dispatcher only logs them: Command::SetSettingsCbor, Reboot, Update, Execute all hit the not yet implemented // TODO(live-cutover) warn arms in crates/podd-core/src/lib.rs:105-113 and go nowhere. So the web UI's reboot button and the generic execute escape hatch are no-ops that look like they worked. (Update overlaps #1; Reboot/Execute/SetSettings do not.) Consider returning a not-implemented status until these are wired, so callers aren't misled.
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
- 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 GoogleBot42/podd
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
GoogleBot42/podd#63 ·
-
documentation
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
GoogleBot42/podd#62 ·
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
GoogleBot42/podd#60 ·
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
GoogleBot42/podd#61 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
GoogleBot42/podd#58 ·
Todos los issues de GoogleBot42/podd
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
rubys/roundhouse#1917 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
[cosmic-applet-battery] Division by zero displays NaN% when UPower reports max_brightness = 0Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
pop-os/cosmic-applets#1619 ·
Los mantenedores suelen responder en 3 días
-
binding:python bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día