Skip to content

[SECENG-364] Pin GitHub Actions to commit SHAs#15

Merged
Stephanie Ginovker (sginovker) merged 5 commits intomasterfrom
security/pin-actions-to-sha
Apr 20, 2026
Merged

[SECENG-364] Pin GitHub Actions to commit SHAs#15
Stephanie Ginovker (sginovker) merged 5 commits intomasterfrom
security/pin-actions-to-sha

Conversation

@sginovker
Copy link
Copy Markdown
Contributor

@sginovker Stephanie Ginovker (sginovker) commented Apr 8, 2026

Ticket

SECENG-364

Summary

Pin all external GitHub Actions to commit SHAs for supply chain security. Internal (hoverinc/) actions are left unpinned.

Pinned Actions

Dependabot

Added/updated dependabot.yml to keep GitHub Actions pinned to the latest SHA with a 7-day update cooldown.

Old action deprecated, Ruby 2.7 gone from runners. Use ruby/setup-ruby
v1.301.0 w/ Ruby 3.1 + built-in bundler cache. Drop redundant cache
and bundle install steps.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Locked nokogiri 1.12.4 requires ruby < 3.1.dev.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Fresh resolve to unblock CI — old lock had dead gems (mimemagic,
nokogiri < 3.1). Deps now resolve cleanly on Ruby 3.4.

Safe to update: Gemfile.lock only affects CI, not consumers. ehi-api
and product-catalog resolve their own deps independently. This gem has
no runtime dependencies — Rails/nokogiri are dev/test only, pulled in
via Combustion for rspec.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
OpenStruct removed from default gems in Ruby 3.4.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sginovker Stephanie Ginovker (sginovker) marked this pull request as ready for review April 20, 2026 17:20
@sginovker Stephanie Ginovker (sginovker) merged commit adf9054 into master Apr 20, 2026
1 check passed
@sginovker Stephanie Ginovker (sginovker) deleted the security/pin-actions-to-sha branch April 21, 2026 20:54
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.

3 participants