webui-auth-edge: auth handler does not fail closed when handle() rejects
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 90/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- aws, javascript
- Lĩnh vực
- authentication
Hướng nghiên cứu
Đọc packs/kirocrew/webui-auth-edge/index.js quanh dòng 167 và theo dõi cách catch bao quanh gửi phản hồi 503. Thay đổi giá trị trả về để một lần từ chối của auth.handle(event) đến được trình xử lý đó, sau đó xác minh rằng lần từ chối tạo ra phản hồi fail closed.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- Shell
- Star
- 73
- Fork
- 11
- Merge trung bình
- 6 giờ 46 phút
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của inceptionstack/lowkey
-
bug good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
inceptionstack/lowkey#135 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
inceptionstack/lowkey#134 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 94/100
inceptionstack/lowkey#133 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
inceptionstack/lowkey#131 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
inceptionstack/lowkey#130 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của inceptionstack/lowkey
Issue tương tự
-
[cifuzz] "Minimizing crashes reduces fuzzing time" is logged for every batch run, even with minimize-crashes offCó thể đã có người làm @martinfrancois đã nhận hôm nay. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
alunduil/alunduil-chezmoi#914 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
cao-evolution cao-evolution:compiler-security
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 86/100
githubnext/gh-aw-cao#17586 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày