Skip to content

GitHub Actions SHA Pinning#42

Merged
mkmn merged 1 commit into
masterfrom
security/sha-pinning-gha
Jun 1, 2026
Merged

GitHub Actions SHA Pinning#42
mkmn merged 1 commit into
masterfrom
security/sha-pinning-gha

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented May 27, 2026

Copy link
Copy Markdown
Contributor

■ やりたいこと

GitHub Actionsをターゲットにしたサプライチェーン攻撃への対策として、GitHub Actions内で呼び出している「外部のアクション」や「共通ワークフロー」のバージョン指定を、安全なコミットSHA(40文字のハッシュ値)に固定する対応を行います。

■ 本PRで実施した内容

  • GitHub Actions呼び出しのSHA固定(ハッシュ化)
  • 既存のワークフローファイル(.github/workflows/*.yml)をスキャンし、未固定のActionをpinactを用いてセキュアなSHAハッシュ値に固定(ピン留め)しました。

■ 各開発チームの皆様へのお願い(レビュー・マージについて)

順次作成されるPRの内容をご確認の上、なるべく早期のマージにご協力をお願いいたします。

  • SHAハッシュ値の細かな検証は不要です
    • 自動生成されたSHAハッシュ値が正しいかどうかの細かな整合性検証は不要です。自動ツール(pinact)による機械的な置換ですので、差分に不審な点(意図しないファイルの変更など)がなければ、そのままマージをお願いいたします。

Co-Authored-By: Yudai SUZUKI <yudai.suzuki@smarthr.co.jp>
@devin-ai-integration devin-ai-integration Bot requested a review from a team as a code owner May 27, 2026 06:54
@devin-ai-integration devin-ai-integration Bot requested review from AtsushiM and removed request for a team May 27, 2026 06:54
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration Bot requested a review from wakasa51 May 27, 2026 06:54
@mkmn mkmn merged commit 61f079d into master Jun 1, 2026
4 checks passed
@mkmn mkmn deleted the security/sha-pinning-gha branch June 1, 2026 01:48
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.

1 participant