coder / coder/backstage-plugins

Coder plugin: clean up error-handling logic

Ouverte
#134 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
bug good first issue
Langage dominant
TypeScript
Étoiles
67
Forks
9
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par localiser la gestion personnalisée de BackstageHttpError du plugin Coder ainsi que la logique qui vérifie les jetons de session potentiellement expirés. Examinez les tests qui ont échoué dans #132, puis vérifiez que la gestion de AxiosError et le chemin du cache de requêtes 401 se comportent comme prévu sans requête supplémentaire ; le travail est considéré comme terminé lorsque le nettoyage est couvert par des tests réussis.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
typescript
Domaine
authentication, developer-experience
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.