webui-auth-edge: auth handler does not fail closed when handle() rejects
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 90/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- aws, javascript
- Domaine
- authentication
Piste de recherche
Lisez packs/kirocrew/webui-auth-edge/index.js autour de la ligne 167 et suivez la manière dont le catch environnant envoie la réponse 503. Modifiez la valeur de retour afin qu’un rejet de auth.handle(event) atteigne ce gestionnaire, puis vérifiez que le rejet produit la réponse de refus par défaut.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
In packs/kirocrew/webui-auth-edge/index.js line 167, the code is return auth.handle(event); with no await.
If handle() rejects, the catch block does not run. Thus the function does not send the 503 "fail closed" response.
Fix: use return await auth.handle(event);.
Found by a static read of the code at commit 1f66a8f. The code was not run.
- Langage dominant
- Shell
- Étoiles
- 73
- Forks
- 11
- Merge moyen
- 1 min
- PR mergées (30 j)
- 2
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de inceptionstack/lowkey
-
bug good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
inceptionstack/lowkey#135 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
inceptionstack/lowkey#134 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 94/100
inceptionstack/lowkey#133 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
inceptionstack/lowkey#131 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
inceptionstack/lowkey#130 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de inceptionstack/lowkey
Issues similaires
-
diagnosing-superpowers: transcript-as-evidence guard never reaches the dispatcher or the analystsOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
obra/superpowers#2662 ·
Les mainteneurs répondent en général sous 3 jours
-
package-update
Difficulté 1/5 Moins d'une heure Accessibilité débutants 69/100
oSoWoSo/oco#844 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
bug linux priority-low
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
Les mainteneurs répondent en général sous 1 jour
-
chart-check misses a config variable whose env call spans linesPeut-être pris @thebadalov l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
kubemg/kubemg#291 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
rossoctl/automation#130 ·
Les mainteneurs répondent en général sous 3 jours