fork されたリポジトリからのプルリクで Actions のシークレットを参照できるようにしたい

odan 2022/08/19

メンテナンスしている OSS にたまにプルリクが飛んでくる
この OSS ではテストの中に GCP/AWS にアクセスするものがある
他人から飛んてきたプルリクでもいい感じにテストを実行したい

odan 2022/08/19

pull_request_target を使えばシークレットにアクセスできるのは知っているけど、任意のプルリクでそれをやるのは怖いので何か承認をはさみたい

なんて考えてたけど GitHub 公式ブログにやり方が解説されていた

日本語の解説も見つかった

odan 2022/08/19

sandbox で試してみた

name: CI

on:
  pull_request:
  pull_request_target:
    types: [labeled]

jobs:
  build:
    runs-on: ubuntu-latest
    if: ${{ github.event_name == 'pull_request' || (github.event_name == 'pull_request_target' && contains(github.event.pull_request.labels.*.name, 'safe to test'))  }}

    steps:
      - uses: actions/checkout@v3
      - run: ./scripts/echo-foo.sh
        env:
          FOO: ${{ secrets.FOO }}

job に対する if で pull_request_target の場合は label を見る感じ

odan 2022/08/19

当然意図したとおりに secret にアクセスできていた

odan 2022/08/19

GitHub で pull_request_target labeled "safe to test" で検索すると色々ヒットする

このリポジトリではイベントは pull_request_target をだけにしている
依存している別の Actions はこれ

'safe to test' がついているプルリクに push があると何でもかんでも実行するっぽい...？
後から攻撃のコードを追加されるとやばそう

odan 2022/08/19

このリポジトリは github.event.pull_request.head.repo.fork == false を見ていて面白い

odan 2022/08/19

こうやってステータスが2つに分かれるのなんとかしたい

odan 2022/08/19

fork が false のとき
- テストを実行
fork が true のとき
- event がラベルの追加である
  - safe to test ラベルが追加されている
    - テストを実行

というのが良さそう？