Run permissions step on pull_request_target but not push (#2548)

getsentry · Nov 30, 2023 · e9b5855 · e9b5855
1 parent cd3f08b
commit e9b5855
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/.github/workflows/test-integration-aws_lambda.yml b/.github/workflows/test-integration-aws_lambda.yml
@@ -32,7 +32,8 @@ jobs:
       - uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # v3.1.0
         with:
           persist-credentials: false
-      - name: permissions
+      - name: Check permissions on PR
+        if: github.event_name == 'pull_request_target'
         run: |
           python3 -uS .github/workflows/scripts/trigger_tests_on_label.py \
               --repo-id ${{ github.event.repository.id }} \
@@ -45,6 +46,9 @@ jobs:
           # these can contain special characters
           ARG_USERNAME: ${{ github.event.pull_request.user.login }}
           ARG_LABEL_NAMES: ${{ toJSON(github.event.pull_request.labels.*.name) }}
+      - name: Check permissions on repo branch
+        if: github.event_name == 'push'
+        run: true
   test-pinned:
     needs: check-permissions
     timeout-minutes: 30

diff --git a/scripts/split-tox-gh-actions/templates/check_permissions.jinja b/scripts/split-tox-gh-actions/templates/check_permissions.jinja
@@ -6,7 +6,8 @@
         with:
           persist-credentials: false
 
-      - name: permissions
+      - name: Check permissions on PR
+        if: github.event_name == 'pull_request_target'
         run: |
           {% raw %}
           python3 -uS .github/workflows/scripts/trigger_tests_on_label.py \
@@ -23,3 +24,7 @@
           ARG_USERNAME: ${{ github.event.pull_request.user.login }}
           ARG_LABEL_NAMES: ${{ toJSON(github.event.pull_request.labels.*.name) }}
           {% endraw %}
+
+      - name: Check permissions on repo branch
+        if: github.event_name == 'push'
+        run: true