补全组件、问题信息命令:审查留下的 P2

オープン
#14 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
25/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
go, powershell
領域
cli

調査の方向性

This is a bundle of review follow-ups across three commands: components (the .part resume and 416 handling, proxy selection, the --json and --accept-license check order), report-config, and collect-info (CRLF output from live/ shell commands, the shared 60-second WMI PowerShell process, the crash/ dedupe that uses filepath.Abs under %TEMP%). Start by reading how the resume logic and the WMI batch are written, and check which existing tests cover the crash/, mas/, live/ and logs/ branches. Splitting the list into separate issues before picking one item would make "done" easier to verify.

索引モデルが issue の本文から書いたものです。

説明

bug P2

#9 的三个命令(components、report-config、collect-info)审查后留下的 P2:真实存在,但有绕过办法或影响面小,有空就修。

补全组件

  • 续传时收到 HTTP 416,会先删掉 .part,再看同一个镜像能不能从头给。如果这个镜像手里的文件本身就短,一份有效的半成品就被扔掉了,下一个镜像得从 0 开始。原脚本是保留 .part 的。
  • 镜像不支持断点续传、续传请求回了 200 和整个文件时,要清空 .part 从 0 计数(脚本的「这个镜像不支持断点续传,从头下载」)。这条路没有测试。
  • 代理选择(环境变量、Windows 系统代理)、接进客户端这两步没有测试;许可协议清单的整体时限和 16 MB 上限也没有测试。
  • --json 时必须给 --accept-license,这条是在解析根目录、检查状态之后,而且只在有组件缺失时才检查;应该和其余参数检查一起,提前到前面。

收集问题信息

  • live/ 里 shell 命令的输出是 LF 换行,脚本是 CRLF(adb shell 在 Windows 上会把换行转成 CRLF)。
  • 所有 WMI 查询放在一个 PowerShell 进程里,共用一个 60 秒的时限。只要有一类慢,所有靠 WMI 的行就都丢了。改成分开查,或者每类单独限时。
  • 两个和脚本逐行对照的测试场景,没有走到 crash/、mas/、live/、logs/ 的几个分支;每一节查询自己出错的情况,也只测了几节。
  • zip 里 crash/ 的条目没有逐字节核对,拷坏了也发现不了。
  • Ctrl+C 那条路(删暂存目录、返回 canceled)没有测试。
  • 崩溃转储那一节靠比较 filepath.Abs 的字符串来去重;%TEMP% 是 8.3 短路径时,用来去重的基准目录会算成两个(已经修了展开 8.3 短名那一层,这里是基准目录这一层)。
主要言語
Go
スター
0
フォーク
1
平均マージ
2時間 42分
マージ済み PR(30日)
13

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

ModAVD/ModAVDManager のほかの issue

ModAVD/ModAVDManager の issue をすべて見る

似ている issue

Go の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。