webui-auth-edge: auth handler does not fail closed when handle() rejects
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 90/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- aws, javascript
- Área
- authentication
Línea de trabajo
Lee packs/kirocrew/webui-auth-edge/index.js alrededor de la línea 167 y sigue cómo el catch que lo rodea envía la respuesta 503. Cambia el valor de retorno para que un rechazo de auth.handle(event) llegue a ese controlador y, después, verifica que el rechazo produzca la respuesta de cierre seguro.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Shell
- Estrellas
- 73
- Forks
- 11
- Merge medio
- 6 h 46 min
- PR fusionados (30 d)
- 5
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de inceptionstack/lowkey
-
bug good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
inceptionstack/lowkey#135 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
inceptionstack/lowkey#134 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 94/100
inceptionstack/lowkey#133 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
inceptionstack/lowkey#131 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
inceptionstack/lowkey#130 ·
Los mantenedores suelen responder en 1 día
Todos los issues de inceptionstack/lowkey
Issues similares
-
Add .gitignorePosiblemente ocupada @ajeanmahoney la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
tesseract-ocr/tesseract#4653 ·
Los mantenedores suelen responder en 1 día
-
complexity::1-low effort::1-hours priority::1-must
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
jumpingrivers/audit.workbench#36 ·
Los mantenedores suelen responder en 1 día
-
area:install bug needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 2 días
-
Duplicate CFBundleIdentifier in copied _Testing_* overlay frameworks breaks simulator installAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
johnno1962/InjectionNext#175 · 2 comentarios ·