Missing safety comment: which variants are upheld?
维护者通常 2 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 48/100
- Issue 类型
- 文档
- 描述清晰度
- 基本清楚
- 活跃度
- 停滞
- 技术栈
- rust
- 领域
- networking
调研方向
从 src/proto/h2/server.rs 第 478 行附近开始，检查 UpgradedSendStream::new。先阅读 Rust API Guidelines 文档链接，然后确定安全性注释应描述哪些不变量，以及构造函数要求哪些不变量。完成的标准是两个位置都清楚地记录这些不变量。
由索引模型根据 Issue 内容生成。
描述
Per Rust API Guidelines, unsafe functions should be documented with invariants:
https://rust-lang.github.io/api-guidelines/documentation.html
During review of the source, it was pointed out there is no safety comment here that describes what invariants are upheld:
https://github.com/hyperium/hyper/blob/90eb95f62a32981cb662b0f750027231d8a2586b/src/proto/h2/server.rs#L478
Also UpgradedSendStream::new does not state which invariants need to be upheld.
Would it be possible to add this?
- 主要语言
- Rust
- 星标
- 16.3k
- 派生
- 1.8k
- 平均合并
- 4 天 10 小时
- 30 天内合并 PR
- 21
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
hyperium/hyper 的其他 Issue
-
C-feature
难度 1/5 1 小时以内 新手友好度 65/100
hyperium/hyper#2652 · 4 个 reaction ·
维护者通常 2 天内回复
-
Write a HIP for `Body::poll_progress()`可能已有人在做 @cratelyn 于 1 天前认领。 未关闭A-body B-rfc C-feature
hyperium/hyper#4228 · 已指派 1 人 ·
维护者通常 2 天内回复
-
难度 3/5 1-2 天 新手友好度 74/100
维护者通常 2 天内回复
-
Upgraded HTTP/2 CONNECT streams cannot be reset, so a failed tunnel looks like a clean close可能已有人在做 @jeremyjpj0916 于 10 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 55/100
维护者通常 2 天内回复
-
hyper-util legacy client: an HTTP/1 request can hang forever when the connection closes while the request is being queued可能已有人在做 @akasakariko 于 3 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 62/100
维护者通常 2 天内回复
相似的 Issue
-
mxl-compile: пример заполнения ячеек отклоняется Unica可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭
难度 1/5 1-3 小时 新手友好度 91/100
IngvarConsulting/unica#1301 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
-
难度 2/5 1-3 小时 新手友好度 82/100
-
难度 1/5 1 小时以内 新手友好度 75/100
ajeetraina/awesome-docker-sbx#220 ·
-
`helios / deploy`: switch zone wait in `deploy.sh` has almost no headroom over healthy startup times可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭Test Flake
难度 2/5 1-3 小时 新手友好度 74/100
oxidecomputer/omicron#11453 ·
维护者通常 1 天内回复