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

GET /organisations/{orgID}/users returns password hash and emailHash

未关闭 适合新手
#966 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

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

调研方向

阅读OrganisationsController.ts并找到使用include: { users: true }的getOrganisationUsers;查看#878和#926中的相关修复以了解背景。添加一个测试,断言此路由的响应中不包含password和emailHash。然后验证用户包含操作会省略这两个字段,并运行相关测试。

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

描述

backend bug

GET /organisations/{orgID}/users returns every column of each user in the organisation, including the stored password hash and emailHash, so any OrganisationAdmin can read them.

getOrganisationUsers in OrganisationsController.ts uses include: { users: true } with no omit. #878 and #926 fixed the same leak on the /users routes, but this route was missed. The handler also logs the full response, which #943 already covers.

Fix: include: { users: { omit: { password: true, emailHash: true } } }, plus a test that asserts both fields are absent. If #937 lands first, its UserResponse[] response type leaves both fields out of the type but not out of the response, so the omit is still needed.

Related: #921, #922, #943, #937.

主要语言
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 摘要。