coder / coder/backstage-plugins

Coder plugin: clean up error-handling logic

Abierto
#134 0 comentarios 0 reacciones 0 asignados Ver en GitHub
bug good first issue
Lenguaje dominante
TypeScript
Estrellas
67
Forks
9
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

The Coder plugin has some vestigial code from before we started using Axios. Now that we have Axios's `AxiosError`, there's far less of a need for the custom `BackstageHttpError`

In addition, the logic for spying on potentially-expired Coder session tokens can be cleaned up. The current code works, but when a Coder query fails, the plugin will make an entirely brand new request to revalidate the auth state. But if the error has a 401 status, we don't need to do that – we know that the session token is likely invalid, so we can mutate the query cache directly.

Tried to get these fixed really quickly in #132, but it started making the tests fail, so I decided to split this off into a separate issue.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza localizando el manejo personalizado de BackstageHttpError del plugin de Coder y la lógica que comprueba los tokens de sesión potencialmente expirados. Revisa las pruebas que fallaron en #132 y, después, verifica que el manejo de AxiosError y la ruta de la caché de consultas 401 se comporten como se espera sin una solicitud adicional; se considerará terminado cuando la limpieza esté cubierta por pruebas satisfactorias.

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

Evaluación

Stack tecnológico
typescript
Área
authentication, developer-experience
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.