Skip to content

Commit

Permalink
Merge pull request linode#279 from lgarber-akamai/fix/acctest-vuln
Browse files Browse the repository at this point in the history
fix: Use full-length SHA refs in testing workflow
  • Loading branch information
lgarber-akamai committed Oct 28, 2022
2 parents 92aa0a5 + b3acae0 commit 99dd6af
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/integration_tests_pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
github.event_name == 'repository_dispatch' &&
github.event.client_payload.slash_command.sha != '' &&
github.event.client_payload.slash_command.tests != '' &&
contains(github.event.client_payload.pull_request.head.sha, github.event.client_payload.slash_command.sha)
github.event.client_payload.pull_request.head.sha == github.event.client_payload.slash_command.sha

steps:
- uses: actions/setup-go@37335c7bb261b353407cff977110895fa0b4f7d8 # pin@v2
Expand All @@ -31,7 +31,7 @@ jobs:
- name: Checkout PR
uses: actions/checkout@v2
with:
ref: 'refs/pull/${{ github.event.client_payload.pull_request.number }}/merge'
ref: ${{ github.event.client_payload.slash_command.sha }}

- run: make ARGS="-run ${{ github.event.client_payload.slash_command.tests }}" fixtures
if: ${{ steps.validate-tests.outputs.match == '' }}
Expand Down

0 comments on commit 99dd6af

Please sign in to comment.