Skip to content

Commit 1bce206

Browse files
Potential fix for code scanning alert no. 3: Workflow does not contain permissions (#43)
Potential fix for [https://github.com/qdequippe-tech/yousign-php-api/security/code-scanning/3](https://github.com/qdequippe-tech/yousign-php-api/security/code-scanning/3) In general, to fix this problem you should explicitly declare a `permissions` block for the workflow and/or each job so that the `GITHUB_TOKEN` has only the privileges required to run. For read-only CI tasks that just check out the code and run analysis, `contents: read` is typically sufficient. For this specific workflow, the `check-cs` job only checks out the repository and runs PHP-CS-Fixer in dry-run mode, so it does not need any write permissions. The simplest and least-intrusive fix is to add a workflow-level `permissions` block (applies to all jobs) near the top of `.github/workflows/ci.yml`, right after the `name: CI` line. Set `contents: read` as the minimal starting point recommended by CodeQL. No imports or additional methods are needed since this is just a YAML configuration change. Concretely: - Edit `.github/workflows/ci.yml`. - Insert a `permissions:` section after line 1 (`name: CI`) with `contents: read`. - Leave the rest of the workflow unchanged so functionality remains identical, but with a restricted `GITHUB_TOKEN`. _Suggested fixes powered by Copilot Autofix. Review carefully before merging._ Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
1 parent 2874319 commit 1bce206

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
name: CI
2+
permissions:
3+
contents: read
24

35
on:
46
push:

0 commit comments

Comments
 (0)