Skip to content

Add checksum validation to install script#794

Merged
devm33 merged 3 commits intomainfrom
devm33/checksum
Dec 17, 2025
Merged

Add checksum validation to install script#794
devm33 merged 3 commits intomainfrom
devm33/checksum

Conversation

@devm33
Copy link
Copy Markdown
Member

@devm33 devm33 commented Dec 16, 2025

SHA256SUMS.txt available in releases starting with 0.0.370-1

$ VERSION=0.0.370-1 ./install.sh
Installing GitHub Copilot CLI...
Downloading from: https://github.com/github/copilot-cli/releases/download/v0.0.370-1/copilot-darwin-arm64.tar.gz
✓ Checksum validated
Notice: Replacing copilot binary found at /Users/devm33/.local/bin/copilot.
✓ GitHub Copilot CLI installed to /Users/devm33/.local/bin/copilot

Warning: /Users/devm33/.local/bin is not in your PATH
Add it to your PATH by adding this line to your shell profile:
  export PATH="$PATH:/Users/devm33/.local/bin"

Installation complete! Run 'copilot help' to get started.

Copilot AI review requested due to automatic review settings December 16, 2025 14:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds SHA256 checksum validation to the install script to verify the integrity of downloaded releases. The validation occurs after downloading the tarball but before extracting it, providing an additional security layer.

Key changes:

  • Downloads SHA256SUMS.txt file from the release and validates the tarball checksum
  • Supports both versioned and latest release URLs
  • Gracefully handles cases where checksums are unavailable or validation tools are missing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@devm33 devm33 requested a review from Copilot December 16, 2025 16:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Member

@sergiou87 sergiou87 left a comment

Choose a reason for hiding this comment

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

:shipit:

@devm33 devm33 merged commit e5349eb into main Dec 17, 2025
3 checks passed
@devm33 devm33 deleted the devm33/checksum branch December 17, 2025 15:42
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.

4 participants