massive-com / massive-com/client-python
Decoding JSON response errors should include the URL
还没有人认领这个 Issue。
- 主要语言
- Python
- 星标
- 1.5k
- 派生
- 362
- PR 合并指标
- 30 天内没有已合并 PR
描述
Is your feature request related to a problem? Please describe.
In base.py, errors when decoding JSON responses don't include the URL:
logger.error("Error decoding json response: %s", e)
This can make it very difficult to troubleshoot corner-case issues.
Describe the solution you'd like
Include the URL. In this case, something like this would be very helpful:
logger.error("Error decoding json response [%s]: %s", full_url, e)
Please identify all such places where URL context is helpful and include the URL.
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 base.py 中的 JSON 响应解码日志开始,然后搜索其他可能需要 URL 上下文的 JSON 解码错误。检查 full_url 在每个位置的可用方式,并更新相关消息以包含它。完成的标准是,受影响的错误日志能够标识响应 URL,同时不改变底层错误处理。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- python
- 领域
- api
- Issue 类型
- 功能
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 冷清
- 描述清晰度
- 基本清楚
- 新手友好度
- 72/100