Skip to content

Commit 52a89a3

Browse files
authored
ci: add missing top level contents permission (#779)
### Description Two workflows in this repo were missing a top-level `permissions: contents: read` block: - `.github/workflows/manual_release_docs.yaml` - `.github/workflows/on_issue.yaml` The other workflows here, as well as the equivalents in `apify-sdk-python` and `crawlee-python`, all set this default at the file level. This applies the principle of least privilege: jobs that need to escalate (e.g. `contents: write`, `pages: write`, `issues: write`) still do so via job-level overrides. ### Changes - Add `permissions: contents: read` at the top level of both workflow files.
1 parent abb8ce7 commit 52a89a3

2 files changed

Lines changed: 6 additions & 0 deletions

File tree

.github/workflows/manual_release_docs.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ on:
1212
required: true
1313
type: string
1414

15+
permissions:
16+
contents: read
17+
1518
env:
1619
NODE_VERSION: 22
1720
PYTHON_VERSION: 3.14

.github/workflows/on_issue.yaml

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

9+
permissions:
10+
contents: read
11+
912
jobs:
1013
label_issues:
1114
name: Add labels

0 commit comments

Comments
 (0)