Skip to content

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

Merged
oioki merged 1 commit into
masterfrom
alextarasov/vuln-2365-command-injection-vulnerability-in-getsentryself-hosted
Aug 6, 2026
Merged

ci: Replace curl-pipe-bash with action-setup-cli for Sentry CLI setup#4451
oioki merged 1 commit into
masterfrom
alextarasov/vuln-2365-command-injection-vulnerability-in-getsentryself-hosted

Conversation

@oioki

@oioki oioki commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

  • The "Setup Sentry CLI" step used curl -sL https://sentry.io/get-cli/ | SENTRY_CLI_VERSION="3.4.3" 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.
  • Same pinned version (3.4.3) preserved via the action's version input.

This is one of two live examples of this new action in use — the other is sentry-fastlane-plugin#529, which is green on all checks.

Test plan

  • Validated action.yaml parses correctly
  • Verify the end-to-end test job installs sentry-cli 3.4.3 correctly on this PR

Fixes VULN-2365

🤖 Generated with Claude Code

Refs VULN-2365
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@linear-code

linear-code Bot commented Aug 5, 2026

Copy link
Copy Markdown

VULN-2365

@oioki
oioki requested a review from BYK August 6, 2026 09:02
@oioki
oioki merged commit ec4fc54 into master Aug 6, 2026
20 checks passed
@oioki
oioki deleted the alextarasov/vuln-2365-command-injection-vulnerability-in-getsentryself-hosted branch August 6, 2026 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants