Skip to content

Set persist-credentials: false on all checkout steps#13

Open
a-know wants to merge 1 commit into
mainfrom
security/persist-credentials-false
Open

Set persist-credentials: false on all checkout steps#13
a-know wants to merge 1 commit into
mainfrom
security/persist-credentials-false

Conversation

@a-know
Copy link
Copy Markdown
Owner

@a-know a-know commented May 27, 2026

Summary

  • actions/checkout のデフォルト設定(persist-credentials: true)では、GitHubトークンをBase64エンコードした認証情報ファイルがランナー上に残り続ける
  • サプライチェーン攻撃やスクリプトインジェクションによってトークンが盗まれるリスクがある
  • 全ワークフローの checkout ステップに persist-credentials: false を追加し、チェックアウト完了後に認証情報ファイルを削除するよう設定

対象ファイル:

  • .github/workflows/ci.yml(2箇所)
  • .github/workflows/deploy_ios.yml(1箇所)

いずれのワークフローも、checkout後にgit操作は行っておらず、この設定変更による動作への影響はない。

Test plan

  • CI ワークフローが正常に通過することを確認

参考: https://zenn.dev/kou_pg_0131/articles/gha-checkout-persist-credentials

🤖 Generated with Claude Code

トークンが認証情報ファイルとしてランナー上に残るリスクを排除するため、
全ワークフローの actions/checkout に persist-credentials: false を設定。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@a-know a-know force-pushed the security/persist-credentials-false branch from 45d5528 to 096f1ed Compare May 27, 2026 13:00
@a-know
Copy link
Copy Markdown
Owner Author

a-know commented May 28, 2026

/e2e

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