Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

GET /users/admin leaks other admins' study assignments via adminOfStudies field

未关闭
#922 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
45/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
冷清
技术栈
typescript
领域
security

调研方向

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.

由索引模型根据 Issue 内容生成。

描述

backend bug question

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.

主要语言
TypeScript
星标
4
派生
0
平均合并
14 小时 50 分钟
30 天内合并 PR
5

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

Garvan-Data-Science-Platform/ctrl 的其他 Issue

查看 Garvan-Data-Science-Platform/ctrl 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。