Skip to content

ci: Replace curl-pipe-bash with action-setup-cli for Sentry CLI setup - #508

Merged
oioki merged 2 commits into
mainfrom
alextarasov/vuln-2227-command-injection-vulnerability-in-getsentryuptime-checker
Aug 6, 2026
Merged

ci: Replace curl-pipe-bash with action-setup-cli for Sentry CLI setup#508
oioki merged 2 commits into
mainfrom
alextarasov/vuln-2227-command-injection-vulnerability-in-getsentryuptime-checker

Conversation

@oioki

@oioki oioki commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary

  • The "Install sentry-cli" step used curl -sL https://sentry.io/get-cli/ | bash, piping a remote install script directly into a shell.
  • Replaced it with getsentry/action-setup-cli, which downloads the sentry-cli release asset for the runner's OS/arch and verifies its sha256 against the digest GitHub recorded for that asset via the Releases API, before adding it to PATH. Uses gh only, no curl.

Test plan

  • Validated image.yml parses correctly
  • Verify the build-image-production job still installs sentry-cli and uploads debug symbols correctly on this PR

Fixes VULN-2227

🤖 Generated with Claude Code

Refs VULN-2227
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@oioki
oioki requested review from a team as code owners August 6, 2026 09:18
@linear-code

linear-code Bot commented Aug 6, 2026

Copy link
Copy Markdown

VULN-2227

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e89358b. Configure here.

Comment thread .github/workflows/image.yml
@oioki
oioki merged commit c397c8d into main Aug 6, 2026
19 checks passed
@oioki
oioki deleted the alextarasov/vuln-2227-command-injection-vulnerability-in-getsentryuptime-checker branch August 6, 2026 10:37
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