Skip to content

Commit 8795229

Browse files
committed
fix: add explicit permissions to GitHub workflows
Address security warnings by adding explicit GITHUB_TOKEN permissions: - ci-ts.yaml: contents: read (minimal permissions for CI) - publish.yml: contents: read, packages: write (for npm publishing) This follows GitHub security best practices to limit token permissions.
1 parent ba4634e commit 8795229

2 files changed

Lines changed: 7 additions & 0 deletions

File tree

.github/workflows/ci-ts.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ concurrency:
1212
group: ${{ github.workflow }}-${{ github.event.pull_request.head.ref || github.ref }}
1313
cancel-in-progress: true
1414

15+
permissions:
16+
contents: read
17+
1518
jobs:
1619
ts-checks:
1720
uses: zondax/_workflows/.github/workflows/_checks-ts.yaml@main

.github/workflows/publish.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@ concurrency:
1414
group: ${{ github.workflow }}-${{ github.ref }}
1515
cancel-in-progress: false
1616

17+
permissions:
18+
contents: read
19+
packages: write
20+
1721
jobs:
1822
publish-npm:
1923
uses: zondax/_workflows/.github/workflows/_publish-npm.yaml@main

0 commit comments

Comments
 (0)