Skip to content

[codex] Verify upgrade installer checksum - #3295

Merged
Widthdom merged 1 commit into
mainfrom
fix-issue3004
Jun 5, 2026
Merged

[codex] Verify upgrade installer checksum#3295
Widthdom merged 1 commit into
mainfrom
fix-issue3004

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Publish install.sh as a release asset so sha256sums.txt covers the installer script.
  • Make cdidx upgrade download release sha256sums.txt, verify the install.sh SHA-256 entry, and only then execute the installer.
  • Document the checksum trust boundary and add a security changelog fragment.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~ProgramRunnerTests"
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • dotnet test ran on final HEAD: net8 passed; net9 had one unrelated ExtractorPluginRegistryTests.LoadPluginAssemblies_RetainsCapDiagnosticWhenCandidatesAlsoFailToLoad Regex timeout. The failing test passed when rerun directly with the same filter on net8/net9.

Documentation and Changelog

  • Updated README.md.
  • Added changelog.d/unreleased/3004.security.md.

Review

  • Adversarial review: No blocking/actionable issues found.

Fixes #3004

@Widthdom
Widthdom marked this pull request as ready for review June 5, 2026 07:18
@Widthdom
Widthdom merged commit 069743c into main Jun 5, 2026
11 checks passed
@Widthdom
Widthdom deleted the fix-issue3004 branch June 5, 2026 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

upgrade should verify downloaded installer script integrity

1 participant