Skip to content

security: pin GitHub Actions to SHA hashes#3272

Merged
alfondotnet merged 1 commit into
mainfrom
security/pin-actions-to-sha
Mar 25, 2026
Merged

security: pin GitHub Actions to SHA hashes#3272
alfondotnet merged 1 commit into
mainfrom
security/pin-actions-to-sha

Conversation

@alfondotnet

@alfondotnet alfondotnet commented Mar 24, 2026

Copy link
Copy Markdown
Member

Summary

  • Pin all uses: references in GitHub Actions workflows to full SHA hashes
  • Prevents supply chain attacks via tag mutation or typosquatting

Context

Test plan

  • Verify CI passes with pinned references
  • Spot-check that pinned SHAs match expected release tags

Note

Low Risk
Low risk: workflow uses: references are pinned from tags to commit SHAs, reducing supply-chain exposure without changing the workflows' logic.

Overview
Pins reusable workflow/action references in .github/workflows/claude.yml, .github/workflows/issue-notifications.yml, and .github/workflows/lock.yml from floating version tags (e.g., v4, v2, v2.0.1) to specific commit SHAs.

This hardens CI automation against upstream tag mutation/typosquatting while keeping the referenced versions the same (noted via inline comments).

Written by Cursor Bugbot for commit e262ecd. This will update automatically on new commits. Configure here.

Pin all third-party GitHub Actions to full SHA references
to prevent supply chain attacks via tag manipulation or
typosquatting.

Ref: https://rosesecurity.dev/2026/03/20/typosquatting-trivy.html
@alfondotnet alfondotnet requested a review from a team as a code owner March 24, 2026 23:21
@codecov

codecov Bot commented Mar 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.38%. Comparing base (c6f7c10) to head (e262ecd).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3272   +/-   ##
=======================================
  Coverage   79.38%   79.38%           
=======================================
  Files         357      357           
  Lines       14347    14347           
  Branches     1959     1959           
=======================================
  Hits        11389    11389           
  Misses       2154     2154           
  Partials      804      804           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

alfondotnet commented Mar 25, 2026

Copy link
Copy Markdown
Member Author

Merge activity

  • Mar 25, 4:05 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 25, 4:05 PM UTC: @alfondotnet merged this pull request with Graphite.

@alfondotnet alfondotnet merged commit 097ce1b into main Mar 25, 2026
35 checks passed
@alfondotnet alfondotnet deleted the security/pin-actions-to-sha branch March 25, 2026 16:05
github-merge-queue Bot pushed a commit that referenced this pull request Mar 26, 2026
**This is an automatic release.**

## RevenueCat SDK
### 🐞 Bugfixes
* [EXTERNAL] fix: ensure activity is attached before showing in-app
messages (#3274) contributed by @matteinn (#3275) via Toni Rico
(@tonidero)
* Ensure MediaPlayer has dedicated thread owner that is not the main
thread (#3148) via Jacob Rakidzich (@JZDesign)
* Fix heartbeat monitor and Slack notifications for nightly integration
tests (#3259) via Rick (@rickvdl)

## RevenueCatUI SDK
### Paywallv2
#### ✨ New Features
* Feat: Restore gating in paywalls UI (#3171) via Jacob Rakidzich
(@JZDesign)

### 🔄 Other Changes
* security: pin GitHub Actions to SHA hashes (#3272) via Alfonso
Embid-Desmet (@alfondotnet)
* Bump activesupport from 8.0.2 to 8.0.4.1 (#3270) via dependabot[bot]
(@dependabot[bot])
* Merge release PR after deploy (#3269) via Antonio Pallares
(@ajpallares)
* Require PR approval before release tagging (#3268) via Antonio
Pallares (@ajpallares)
* Bump json from 2.18.1 to 2.19.2 (#3261) via dependabot[bot]
(@dependabot[bot])
* feat(ads): update admob sample app (#3264) via Peter Porfy
(@peterporfy)
* feat(ads): add vanilla-ad-tracker-sample (#3263) via Peter Porfy
(@peterporfy)
* [Purchase Tester]: Persist appUserId on login screen across app
launches (#3266) via Will Taylor (@fire-at-will)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Mostly release automation and versioning changes, but it modifies
CI/orb references and deploy/merge automation, which could affect the
release pipeline if misconfigured.
> 
> **Overview**
> Cuts the `9.28.0` release by removing `-SNAPSHOT` across version
sources (`.version`, `gradle.properties`, `Config.frameworkVersion`) and
updating sample/test-app dependency pins to `9.28.0`.
> 
> Updates release documentation: publishes Dokka docs to the `9.28.0` S3
path, updates `docs/index.html` redirect to `9.28.0`, and rolls
`CHANGELOG.latest.md` into a new `9.28.0` section in `CHANGELOG.md`.
> 
> Tweaks release tooling/CI: pins `fastlane-plugin-revenuecat_internal`
to a specific git ref (and bumps a few Ruby deps), switches the CircleCI
`revenuecat/sdks-common-config` orb to a dev commit, and adds a
temporary `test_merge_queue` workflow to exercise
`revenuecat/merge-release-pr` with `use_merge_queue: true`.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
5050888. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants