Skip to content

feat: verify Helm downloads and support fallback mirrors - #315

Open
David Gamero (davidgamero) wants to merge 3 commits into
Azure:mainfrom
davidgamero:verify-downloads-and-fallback
Open

David Gamero (davidgamero) wants to merge 3 commits into
Azure:mainfrom
davidgamero:verify-downloads-and-fallback

Conversation

@davidgamero

Copy link
Copy Markdown
Collaborator

Summary

Security

Checksums are downloaded from the same base URL as their archive. Missing, malformed, or mismatched checksums fail closed. When a fallback mirror is configured, both its archive and checksum must download successfully and match before extraction.

Validation

  • npm test (62 passing)
  • npm run typecheck
  • npm run format-check
  • npm run build
  • npm audit --audit-level=low (0 vulnerabilities)
  • git diff --check

actionlint still reports the repository's existing missing lib/index.js source-tree warning and deprecated integration-test token input; neither is introduced by this change.

@davidgamero
David Gamero (davidgamero) requested a review from a team as a code owner November 4, 2026 19:44

@bosesuneha Suneha Bose (bosesuneha) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documenting that this changes v5 behavior: fresh downloads require matching .sha256 files. Archive-only custom mirrors must add them.

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.

2 participants