Rate limit unauthenticated /password/generate-reset-link
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 52/100
- Issue 类型
- 功能
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- typescript
调研方向
Start in UsersController at POST /users/password/generate-reset-link, then read the /auth/login code that uses retriesRemaining and lockedUntil as the stated pattern. The issue proposes both per-IP throttling and a per-user cooldown, but does not name files or tests; locate the relevant tests and run them. Done means repeated requests are limited by both IP and user without changing normal reset-link behavior.
由索引模型根据 Issue 内容生成。
描述
POST /users/password/generate-reset-link in UsersController is @NoSecurity() and fires a mail send on every request that matches a real user. No per-IP throttle. No per-user cooldown.
Two things this leaves open:
- Someone who knows a real user's email can mail-bomb the inbox with valid reset links (15-minute TTL each, each writes a
PasswordResetTokenrow). - Someone spamming the endpoint with random emails triggers a mail per matched real user at no cost.
/auth/login already has a per-user retry lock via retriesRemaining and lockedUntil. Same pattern isn't applied here.
Suggested fix
Add both:
- Per-IP throttle via
express-rate-limitor similar — blocks a script hammering the endpoint with random emails regardless of validity. - Per-user cooldown matching the
/auth/loginretry-lock pattern — blocks a distributed attack rotating IPs against one real user's email.
- 主要语言
- TypeScript
- 星标
- 4
- 派生
- 0
- 平均合并
- 14 小时 50 分钟
- 30 天内合并 PR
- 5
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
Garvan-Data-Science-Platform/ctrl 的其他 Issue
-
难度 1/5 1-3 小时 新手友好度 82/100
-
admin-client backend bug
难度 2/5 1-3 小时 新手友好度 76/100
-
backend bug
难度 2/5 1-3 小时 新手友好度 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 条评论 ·
-
backend bug
难度 2/5 1-3 小时 新手友好度 72/100
-
backend bug
难度 2/5 1-3 小时 新手友好度 78/100
查看 Garvan-Data-Science-Platform/ctrl 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 74/100
QuantumNous/new-api#7839 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 70/100
维护者通常 1 天内回复
-
bug p2 triaged
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
test: make the #3408 Claude OAuth token fixture obviously fake so secret scanners stop flagging it未关闭
难度 2/5 1-3 小时 新手友好度 76/100
code-yeongyu/senpi#3436 ·
维护者通常 1 天内回复
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
难度 1/5 1 小时以内 新手友好度 78/100
lingdojo/kana-dojo#32505 · 1 条评论 · 5 个 reaction ·
维护者通常 1 天内回复