Skip to content

Commit 44965d6

Browse files
authored
ci(svf-validation): declare permissions: {contents: read} (#222)
The other 16 workflows in .github/workflows/ declare a top-level permissions block; svf-validation.yml was missed. Bring it in line with the sibling convention. Today the job runs first-party actions only (actions/checkout, actions/setup-python) plus 'make validate-svf-contracts', so an over-broad default token has no direct exploit path. The failure mode being closed off is future drift: a later step adds a third-party action, or make invokes a script that ends up with GITHUB_TOKEN, and the workflow inherits the repo default of read+write on contents/PRs/ issues instead of the explicit read this file should always have had.
1 parent e3c5357 commit 44965d6

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

.github/workflows/svf-validation.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ on:
66
pull_request:
77
workflow_dispatch:
88

9+
permissions:
10+
contents: read
11+
912
jobs:
1013
sourceos-svf-contracts:
1114
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)