补全组件、问题信息命令:审查留下的小问题(P3)

未关闭
#15 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
24/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
活跃
技术栈
go, powershell
领域
cli

调研方向

This is a batch of unrelated P3 leftovers, so pick one item and treat the rest as separate work. Start from the named entry points: asideName for the infinite loop on os.Lstat errors, modavd help for the exit-code text, and --keep-stage for the Ctrl+C cleanup. Done means the chosen item has a test that reproduces the problem and passes after the fix.

由索引模型根据 Issue 内容生成。

描述

P3

#9 审查留下的 P3,闲了再修:

  • 补全组件:
    • asideName 在 os.Lstat 一直返回别的错误时会一直循环;
    • 旧目录改名加 -2、-3 后缀那条只是碰巧被测到;
    • 非 2xx 响应有没有关掉,没有测试;
    • 重试间隔的 3 秒和响应取消,没有测试;
    • 后一个组件失败时,--json 的错误里没写前一个组件的旧目录挪到哪了;
    • 只用一个镜像时,留下一个大小对、SHA-1 不对的 .part,报错会怪到镜像头上;
    • 只设了 NO_PROXY 时会忽略 Windows 系统代理;
    • 超时参数先乘再比较,极大的值会溢出;
    • 几处和脚本对齐的细节没有测试(zip 里重复的条目、空间检查扣掉已下载的部分、日志措辞);
    • check 和 components status 共用的组件状态映射,没有逐个字段核对。
  • modavd help 的退出码说明没更新:2 也表示 license_declined,9 没列出来。
  • 收集问题信息:
    • logcat 的 service 字符串和新版 adb.exe 发的不完全一样;
    • zip 条目的时间存成了 UTC,脚本存的是本地时间;
    • 盘的类型那个查询在取盘符失败时会留下一个 PowerShell 子进程;
    • 加了 --keep-stage 时,Ctrl+C 照样会删掉暂存目录;
    • 根目录和暂存目录给的是 8.3 短路径时,没有像脚本那样展开。
主要语言
Go
星标
0
派生
1
平均合并
2 小时 42 分钟
30 天内合并 PR
13

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

ModAVD/ModAVDManager 的其他 Issue

查看 ModAVD/ModAVDManager 的全部 Issue

相似的 Issue

更多 Go Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。