Report helpful error message for missing "jwt" dependency

Abierto
#526 5 comentarios 5 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
48/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
python
Área
authentication

Línea de trabajo

Empieza siguiendo la llamada a JWTAuth.from_settings_file que se muestra en el informe e inspecciona cómo se carga la dependencia opcional jwt. Reproduce el caso de dependencia ausente sin instalar boxsdk[jwt], después haz que el error sugiera pip install boxsdk[jwt] y verifica que la ruta JWT existente siga funcionando cuando la dependencia esté instalada.

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

Descripción

enhancement
Is your feature request related to a problem? Please describe.

As is known from closed issue (https://github.com/box/box-python-sdk/issues/387) if you don't pip install "boxsdk[jwt]", and use the JWT authorization, you get a somewhat confusing message:

  File "/Users/first.last/workspace/repo_name/box/../lib/box.py", line 70, in get_jwt_client
    auth = boxsdk.JWTAuth.from_settings_file(load_config(self.app_name))
AttributeError: 'NoneType' object has no attribute 'from_settings_file'
Describe the solution you'd like

Return an error message instead that at least suggests "Did you try pip install boxsdk[jwt]?" when JWTAuth isn't going to work.

Additional context

Using sdk version boxsdk 2.9.0

Lenguaje dominante
Python
Estrellas
460
Forks
223
Merge medio
14 h 18 min
PR fusionados (30 d)
21

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 box/box-python-sdk

Todos los issues de box/box-python-sdk

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.