test : add unit tests for ScanRateLimiter helpers in rate_limiter
@aaniya22 已经在做这个了。
开始于 2026年7月29日。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 35/100
调研方向
Start with backend/secuscan/rate_limiter.py and read the existing unit test conventions under testing/backend/unit/. Add testing/backend/unit/test_rate_limiter_scan_rate_limiter.py to cover the listed helpers and check() cases, then run that test file. Done means each specified behavior and factory argument is verified; the issue says the work is already being handled by tmdeveloper007.
由索引模型根据 Issue 内容生成。
描述
Summary of What Needs to be Done:
The backend/secuscan/rate_limiter.py module (containing ScanRateLimiter, RateLimitExceeded, make_scan_rate_limiter) has no dedicated test file. Add unit tests covering the core helper methods.
Changes that Need to be Made:
- Add a new test file:
testing/backend/unit/test_rate_limiter_scan_rate_limiter.py - Test
ScanRateLimiter._get_client_ip(): X-Forwarded-For, X-Real-IP, direct client.host, None client - Test
ScanRateLimiter._make_key(): key namespace construction - Test
ScanRateLimiter.reset(): clears fallback history and redis_failed flag - Test
ScanRateLimiter.check()withrate_limit=0(early return) - Test
ScanRateLimiter.check()withredis_client=None(in-memory fallback path) - Test
ScanRateLimiter.check()per-minute and per-hour limit enforcement - Test
make_scan_rate_limiterfactory preserves all arguments
Impact that it would Provide:
- Reliability: Rate limiter IP extraction, key generation, and fallback logic are verified
- Security: Ensures rate limiting cannot be bypassed
- Testing coverage: Fills a gap in the untested rate_limiter module
Note: This task is being handled by tmdeveloper007 — please assign to that account when picking it up.
- 主要语言
- Python
- 星标
- 36
- 派生
- 234
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
utksh1/SecuScan 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 76/100
-
area:frontend good first issue level:intermediate priority:low type:testing
难度 2/5 1-3 小时 新手友好度 83/100
-
area:frontend good first issue level:intermediate priority:low type:testing
难度 2/5 1-3 小时 新手友好度 78/100
-
area:frontend good first issue level:beginner priority:low type:testing
难度 2/5 1-3 小时 新手友好度 76/100
-
area:frontend good first issue level:intermediate priority:low type:testing
难度 2/5 1-3 小时 新手友好度 82/100
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 83/100
维护者通常 1 天内回复
-
comp/desktop P3 type/feature
难度 2/5 1-3 小时 新手友好度 78/100
NousResearch/hermes-agent#154285 ·
维护者通常 1 天内回复
-
docs: qualify copied and custom kubectl debug policy notes可能已有人在做 @nym01 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 78/100
kubescape/cel-admission-library#166 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
SACGF/variantgrid#2804 · 1 条评论 ·
维护者通常 1 天内回复
-
[BUG] HTTP lifecycle dispatch treats a NO_PROXY-only environment as corporate egress and skips the resolve-and-pinned session可能已有人在做 @bodapatisaikrishna 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复