webui-auth-edge: auth handler does not fail closed when handle() rejects
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Anfängerfreundlichkeit
- 90/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- aws, javascript
- Bereich
- authentication
Rechercherichtung
Lies packs/kirocrew/webui-auth-edge/index.js ungefähr ab Zeile 167 und verfolge, wie der umgebende catch die 503-Antwort sendet. Ändere den Rückgabewert so, dass eine abgelehnte auth.handle(event) diesen Handler erreicht, und überprüfe anschließend, dass die Ablehnung zur Fail-Closed-Antwort führt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Shell
- Sterne
- 73
- Forks
- 11
- Ø Merge
- 1 Min.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus inceptionstack/lowkey
-
bug good first issue
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
inceptionstack/lowkey#135 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
inceptionstack/lowkey#134 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug good first issue
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 94/100
inceptionstack/lowkey#133 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
inceptionstack/lowkey#131 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
inceptionstack/lowkey#130 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in inceptionstack/lowkey
Ähnliche Issues
-
`openssh_authorized_keys_url` should be moved out of tool-versions.ymlEvtl. vergeben @lemeurherve hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
jenkins-infra/packer-images#3228 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
richardcase/skills#26 ·
Maintainer antworten meist innerhalb von 1 Tag
-
marmite skill: `curl ... | sh` is the first install method, bypassing review and integrity checksOffen
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 78/100
rochacbruno/marmite#565 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
bigbluebutton/docker#406 ·