maintenance-pool::sweep has no test proving it requires the admin's authorization
维护者通常 1 天内回复
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 30/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- rust
- 领域
- testing-qa
调研方向
Start in contracts/maintenance-pool/src/test.rs and compare the targeted authorization mocks used for upgrade with the tests that call sweep under env.mock_all_auths(). Add positive coverage for admin authorization and negative coverage for another address, asserting the host auth error; the tests should fail if sweep does not require the admin's authorization. A linked open pull request (#580) indicates work may already be underway.
由索引模型根据 Issue 内容生成。
描述
Problem
Every maintenance-pool test that calls sweep runs under env.mock_all_auths(), which authorises every address. No test uses targeted env.mock_auths(&[MockAuth { address: <admin>, invoke: … fn_name: "sweep" … }]) or checks that a call signed by a different address fails. (In contracts/maintenance-pool/src/test.rs, targeted mocks exist only for upgrade.) If the require_auth() call were deleted or pointed at the wrong address, the suite would still pass. sweep moves arbitrary 'surplus' to any recipient.
Suggested fix
Add a positive test that mocks only the admin's auth for sweep, and a negative test that mocks another address's auth and asserts the host auth error.
Related: #50
- 主要语言
- Rust
- 星标
- 1
- 派生
- 102
- 平均合并
- 4 小时 12 分钟
- 30 天内合并 PR
- 30
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
MergeFi/contracts 的其他 Issue
-
allocations map retains released issues — phantom ledger drift可能重新可做 @PINYOPATTANAWASANPORN 于 12 天前认领，目前没有进行中的 PR。 未关闭
难度 2/5 1-3 小时 新手友好度 25/100
MergeFi/contracts#608 · 3 条评论 ·
维护者通常 1 天内回复
-
escrow::contribute and milestones::contribute accept new funds after deadline + GRACE_PERIOD, when anyone can already refund or cancel可能已有人在做 @urielit1313-creator 于 29 天前认领。 未关闭bug
难度 3/5 1-2 天 新手友好度 25/100
MergeFi/contracts#568 · 3 条评论 ·
维护者通常 1 天内回复
-
Makefile deploy targets deploy the unoptimised .wasm — `stellar contract optimize` writes *.optimized.wasm可能已有人在做 @urielit1313-creator 于 29 天前认领。 未关闭bug good first issue
难度 2/5 1-3 小时 新手友好度 25/100
MergeFi/contracts#567 · 1 条评论 ·
维护者通常 1 天内回复
-
keep_alive never refreshes instance storage, so a contract kept alive only via keep_alive can still archive Admin/Oracle/Treasury可能已有人在做 @urielit1313-creator 于 29 天前认领。 未关闭bug
难度 4/5 3-5 天 新手友好度 25/100
MergeFi/contracts#566 · 3 条评论 ·
维护者通常 1 天内回复
-
[Regression on closed #254] milestones::deallocate still has no test asserting admin authorization可能已有人在做 @urielit1313-creator 于 29 天前认领。 未关闭security testing
难度 2/5 1-3 小时 新手友好度 25/100
MergeFi/contracts#565 · 1 条评论 ·
维护者通常 1 天内回复
查看 MergeFi/contracts 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 72/100
aws/aws-emulator#30 · 1 条评论 ·
-
area:post bug effort:low P2 source:agent
难度 1/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
Chat images never show without ImageMagick: ffprobe fallback uses an option string ffprobe 9 rejects未关闭
难度 2/5 1-3 小时 新手友好度 79/100
eliasstravik/herdr-projects#1011 ·
-
design-data-spec
难度 2/5 1-3 小时 新手友好度 78/100
adobe/spectrum-design-data#1744 ·
维护者通常 1 天内回复