matrix-org / matrix-org/matrix-python-common
Factor out `SrvResolver`
还没有人认领这个 Issue。
- 主要语言
- Python
- 星标
- 4
- 派生
- 7
- PR 合并指标
- 30 天内没有已合并 PR
描述
Sydent has an SrvResolver class here: https://github.com/matrix-org/sydent/blob/8331f94e1e22c72c84683c271142f3237c707a27/sydent/http/srvresolver.py#L104
Synapse [has one too](https://github.com/matrix-org/synapse/blob/0c3565da4cdbe53646ae0bc737900526a1d3df67/synapse/http/federation/srv_resolver.py#L98-L97) with an associated [test case](https://github.com/matrix-org/synapse/blob/4b965c862dc66c0da5d3240add70e9b5f0aa720b/tests/http/federation/test_srv_resolver.py#L30).
I couldn't see one in Sygnal.
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
调研方向
比较 sydent/http/srvresolver.py 与 synapse/http/federation/srv_resolver.py 及其在 tests/http/federation/test_srv_resolver.py 中的测试。确定应如何为 Sygnal 将 resolver 拆分到 matrix-python-common 中，然后添加相应的测试覆盖；当 Sygnal 能够使用共享 resolver 而无需自己的重复实现时，即视为完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- python
- 领域
- networking
- Issue 类型
- 重构
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100