Uncaught exception when a component has a property whose getter raises (intended?)

Abierto
#672 5 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
58/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
python
Área
cli

Línea de trabajo

Comienza con el reproductor app.py proporcionado y sigue la ruta de enumeración de miembros utilizada por la invocación sin argumentos y por --help. Compara esa ruta con la llamada directa a greet y, después, añade cobertura para una propiedad que lanza una excepción; se considera terminado cuando help y la invocación sin argumentos gestionan el fallo del getter según el comportamiento elegido, mientras greet sigue funcionando.

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

Descripción

When a component has a @property whose getter raises, fire.Fire(component) crashes with the property's raw traceback on bare invocation and on --help, instead of showing usage/help. A direct call to an unrelated method works fine.

Repro (fire 0.7.1):

import fire

class App:
    @property
    def status(self):
        raise RuntimeError("backend unavailable")  # e.g. a lazy / fallible getter

    def greet(self, who="world"):
        return f"hi {who}"

if __name__ == "__main__":
    fire.Fire(App())

$ python app.py --help
  ...
    value = getter(object, key)
  File "app.py", line 5, in status
    raise RuntimeError("backend unavailable")
RuntimeError: backend unavailable

$ python app.py greet        # works
hi world

Fire enumerates members (and reads property values) during help/listing, so any property getter that can raise (lazy config, DB or network access, etc.) takes down --help and bare invocation, even though those properties are never invoked.

Is this intended, or should Fire surface a clean error (or skip properties that raise) during member enumeration? Happy to send a PR if a fix would be welcome.

Found via automated analysis and confirmed with the reproducer above on fire 0.7.1.

Lenguaje dominante
Python
Estrellas
28.2k
Forks
1.5k
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

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 google/python-fire

Todos los issues de google/python-fire

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.