GET /users/admin leaks other admins' study assignments via adminOfStudies field
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 45/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- typescript
- Lĩnh vực
- security
Hướng nghiên cứu
Start by tracing the GET /users/admin handler and its security decoration, then compare how it scopes adminOfStudies with the behavior of GET /studies. The report shows that a StudyAdmin can see another admin's study assignments; first determine whether this is intended, especially for multi-tenant deployments. Done means the intended access behavior is established and the issue is resolved accordingly.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
GET /users/admin returns each admin's adminOfStudies array (with study id and name) in the response, and StudyAdmins can call this endpoint per the security decoration.
Logged in as Emily (StudyAdmin of Short Study only), the response included Jane Smith with adminOfStudies: [{ id: 2, name: "Seed Study" }, { id: 1, name: "Short Study" }]. So Seed Study leaks to Emily via Jane's array, even though GET /studies correctly hides it from her scope.
Filing as a question - is this intended behaviour? Might matter more in multi-tenant deployments since CTRL supports multiple Organisations per instance.
Related to #921.
- Ngôn ngữ chính
- TypeScript
- Star
- 4
- Fork
- 0
- Merge trung bình
- 14 giờ 50 phút
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
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 Garvan-Data-Science-Platform/ctrl
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 82/100
-
admin-client backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 bình luận ·
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Tất cả issue của Garvan-Data-Science-Platform/ctrl
Issue tương tự
-
bnv-manager-v2: accept_invite creates the Mailcow domain admin before the commitCó thể đã có người làm @myOmikron đã nhận hôm nay. Đang mởbug ready-for-agent
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Windows: mio 1.2.3 named-pipe use-after-free panics the native IPC broker and hangs every toolĐang mởplatform:windows tool triaged wontfix
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
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 72/100
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 82/100
nicolettas-muggelbude/RechnungsFee#448 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Android bug feature/error-tracking react-native team/error-tracking
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
PostHog/posthog-js#5310 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày