Skip to content

chore: pin GitHub Actions to commit SHAs [PinnR]#44

Merged
j-sm-n merged 5 commits into
mainfrom
pinnR/GHA-Update-2026-04-15
Apr 28, 2026
Merged

chore: pin GitHub Actions to commit SHAs [PinnR]#44
j-sm-n merged 5 commits into
mainfrom
pinnR/GHA-Update-2026-04-15

Conversation

@kjwangez
Copy link
Copy Markdown
Contributor

@kjwangez kjwangez commented Apr 16, 2026

MX-1360

PinnR: GitHub Actions Security Update

This PR pins GitHub Actions to specific commit SHAs to improve supply chain security.

Why?

Floating tags like v3 or branches like main can be moved to point to different commits, potentially introducing malicious code. Pinning to SHAs ensures the exact code version is used.

Changes

File Action Change
ci.yml actions/checkout v4de0fac2e4500dabe0009e67214ff5f5447ce83dd (v6.0.2)
ci.yml ruby/setup-ruby v17372622e62b60b3cb750dcd2b9e32c247ffec26a (v1.302.0)
ci.yml actions/checkout v4de0fac2e4500dabe0009e67214ff5f5447ce83dd (v6.0.2)
ci.yml ruby/setup-ruby v17372622e62b60b3cb750dcd2b9e32c247ffec26a (v1.302.0)
codeql.yml actions/checkout v3de0fac2e4500dabe0009e67214ff5f5447ce83dd (v6.0.2)
codeql.yml github/codeql-action/init v265216971a11ded447a6b76263d5a144519e5eee1 (codeql-bundle-v2.25.2)
codeql.yml github/codeql-action/autobuild v265216971a11ded447a6b76263d5a144519e5eee1 (codeql-bundle-v2.25.2)
codeql.yml github/codeql-action/analyze v265216971a11ded447a6b76263d5a144519e5eee1 (codeql-bundle-v2.25.2)
dependency-review.yml actions/checkout v3de0fac2e4500dabe0009e67214ff5f5447ce83dd (v6.0.2)
dependency-review.yml actions/dependency-review-action v32031cfc080254a8a887f58cffee85186f0e49e48 (v4.9.0)
release.yml ruby/setup-ruby v17372622e62b60b3cb750dcd2b9e32c247ffec26a (v1.302.0)

Note

If this PR is closed without merging, the branch pinnR/GHA-Update-2026-04-15 will need to be deleted manually.


🤖 Generated by PinnR

@kjwangez kjwangez requested a review from a team as a code owner April 16, 2026 00:02
jmpage
jmpage previously approved these changes Apr 16, 2026
@thiagopecanha
Copy link
Copy Markdown
Contributor

The test failure here is a real regression caused by pg 1.6.3 (released Dec 29, 2025), not a pre-existing issue. The activerecord_6.1 appraisal has no Gemfile.lock, so bundler always resolves to the latest pg — which is now 1.6.3.

The last passing CI run (Dec 8, 2025) used pg 1.6.2. The failing test is the notify_only listener spec in listener_spec.rb — it expects nil payload, which relies on Ruby's missing-argument behavior when the callback is invoked with no args. pg 1.6.3 appears to change notification payload handling in a way that breaks this.

Fix: pin pg in gemfiles/activerecord_6.1.gemfile:

gem "pg", "< 1.6.3"
Verified locally — all 6 listener specs pass with this constraint (resolves to pg 1.6.2).

pg 1.6.3 introduced a regression that breaks the notify_only listener spec.
The activerecord_6.1 gemfile has no Gemfile.lock so bundler always resolves
to the latest pg, which is now 1.6.3. Pinning to < 1.6.3 resolves to 1.6.2,
which passes all 6 listener specs.
@j-sm-n j-sm-n merged commit fac63c8 into main Apr 28, 2026
18 checks passed
@j-sm-n j-sm-n deleted the pinnR/GHA-Update-2026-04-15 branch April 28, 2026 17:22
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.

5 participants