Signed copies in Workloads/ and wsl-comfort/ are stale and no longer verify
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
Start with src/tools/check-signed-drift.ps1 and compare its reported roots with Workloads/, wsl-comfort/, and windows-dev-config/. Run Get-AuthenticodeSignature over the release copies and inspect the existing sign pipeline and the history around #105, #107, and #126. Done means the affected copies are current, retain verifiable signatures, and the validation pass catches signature failures.
由索引模型根据 Issue 内容生成。
描述
I ran Get-AuthenticodeSignature over the signed release copies, and most of the files under Workloads/ and wsl-comfort/ come back NotSigned. The windows-dev-config/ copies are fine.
Windows 11 Pro 26H2 (build 26300.9550), pwsh 7.6.6, Get-ChildItem <root> -Recurse -Filter *.ps1 | Get-AuthenticodeSignature | Group-Object Status on a git archive:
| Root | ff7a538 (before #105) |
231c59b (current main) |
|---|---|---|
windows-dev-config/ |
24 × Valid |
32 × Valid |
Workloads/ |
12 × Valid, 5 × HashMismatch |
1 × Valid, 16 × NotSigned, 1 × HashMismatch |
wsl-comfort/ |
1 × Valid |
1 × NotSigned |
I think this comes from #105. It marked the signed copies -text, so git checks them out byte for byte. windows-dev-config/ was re-committed as raw signed bytes in #107 right after, but the files under Workloads/ and wsl-comfort/ were committed back when *.ps1 text eol=crlf applied (#26), so their blobs have LF. Before #105, checkout turned that back into the CRLF the files were signed with. Now it doesn't, and the signatures don't match. The ZIP from GitHub has the same bytes.
The only Valid file under Workloads/ is winui/setup.ps1, which is new in #125 and gets signed together with windows-dev-config/ (#126, #130, #132). The other sixteen haven't changed since before #105. winui/install.ps1 is the odd one: it was re-signed, it has CRLF and a signature block, its body matches src/ after normalization, and it still comes back HashMismatch. The five shims that were HashMismatch before #105 looked the same.
I tried putting the CR back. That alone makes 13 of the 17 Valid again, signed by Microsoft Corporation: dotnet, go, java, powershell, rust, sql, the six in _common/ and wsl-comfort/install.ps1. php, python, typescript and winforms become HashMismatch instead, so those four, winui/install.ps1 and wsl-comfort/install.ps1 need to go through signing again. I guess you'd rather just run the sign pipeline over those two roots anyway, but if a PR that puts the CR back in the twelve is useful in the meantime, I have one ready.
The practical effect: the README points people at .\Workloads\<id>\install.ps1 and .\wsl-comfort\install.ps1. When the download and the extraction keep the mark of the web, under RemoteSigned PowerShell refuses .\Workloads\python\install.ps1 with "is not digitally signed". dev-config.ps1, which still verifies, gets the publisher prompt instead, or just runs if Microsoft is already a trusted publisher. From a clone there is no mark of the web, so the same python script passes the RemoteSigned check and can run as an unsigned script. Under AllSigned the copies without a valid signature are refused everywhere.
wsl-comfort/ is also out of date compared to src/. The root comfort-shell-bootstrap.sh still has alias ls="eza --icons" (#79, fixed in src/ by #80), and install.ps1 and readme.md are behind since #120. src/tools/check-signed-drift.ps1 lists all three, plus Workloads/winappcli/* from #106, which has never had a signed release copy committed. The drift check strips the signature block before comparing, so it can't catch the signature problem. A Get-AuthenticodeSignature pass over the three roots in CI would, which I think is what #34 asked about.
If that sign run also commits the raw bytes like #107 and checks the signatures afterwards, it should cover all of this. Happy to help test.
- 主要语言
- PowerShell
- 星标
- 2.9k
- 派生
- 202
- 平均合并
- 18 小时 49 分钟
- 30 天内合并 PR
- 24
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
microsoft/WindowsDeveloperConfig 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
microsoft/WindowsDeveloperConfig#139 ·
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 90/100
microsoft/WindowsDeveloperConfig#127 ·
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 91/100
microsoft/WindowsDeveloperConfig#124 ·
维护者通常 1 天内回复
-
Add ShowSecondsInClock as part of the setting Explorer settings可能已有人在做 @kilasuit 于 18 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 62/100
microsoft/WindowsDeveloperConfig#100 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 35/100
microsoft/WindowsDeveloperConfig#97 ·
维护者通常 1 天内回复
查看 microsoft/WindowsDeveloperConfig 的全部 Issue
相似的 Issue
-
enhancement
难度 2/5 1-3 小时 新手友好度 72/100
-
难度 2/5 1-3 小时 新手友好度 82/100
influxdata/helm-charts#856 ·
维护者通常 5 天内回复
-
security trivy
难度 2/5 1-3 小时 新手友好度 68/100
kubefleet-dev/kubefleet#944 ·
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 69/100
postmanlabs/newman#3388 ·
-
难度 2/5 1-3 小时 新手友好度 72/100
coollabsio/coolify#12094 · 1 条评论 ·
维护者通常 1 天内回复