Skip to content

Commit d2187fa

Browse files
chitcommitNick Bianchiclaude
authored
fix: add permissions blocks and pin wrangler-action SHA (#35)
- Add explicit permissions (contents: read) to build-ui and get-credentials jobs (CodeRabbit/GHAS feedback) - Pin cloudflare/wrangler-action to commit SHA da0e0dfe (v3) Co-authored-by: Nick Bianchi <nick@jeanarlene.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b708431 commit d2187fa

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,8 @@ jobs:
8585

8686
build-ui:
8787
runs-on: ubuntu-latest
88+
permissions:
89+
contents: read
8890
needs:
8991
- workflow-secret-policy
9092
- secret-scan

.github/workflows/deploy-pages.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@ on:
1111
jobs:
1212
get-credentials:
1313
runs-on: ubuntu-latest
14+
permissions:
15+
contents: read
1416
outputs:
1517
cloudflare_token: ${{ steps.creds.outputs.cloudflare_token }}
1618
account_id: ${{ steps.creds.outputs.account_id }}
@@ -53,7 +55,7 @@ jobs:
5355
working-directory: ui
5456

5557
- name: Deploy to Cloudflare Pages
56-
uses: cloudflare/wrangler-action@v3
58+
uses: cloudflare/wrangler-action@da0e0dfe58b7a431659754fdf3f186c529afbe65 # v3
5759
with:
5860
apiToken: ${{ needs.get-credentials.outputs.cloudflare_token }}
5961
accountId: ${{ needs.get-credentials.outputs.account_id }}

0 commit comments

Comments
 (0)