Skip to content

Potential fix for code scanning alert no. 23: Workflow does not contain permissions#38

Merged
sydneyrenee merged 1 commit intodevfrom
alert-autofix-23
May 4, 2026
Merged

Potential fix for code scanning alert no. 23: Workflow does not contain permissions#38
sydneyrenee merged 1 commit intodevfrom
alert-autofix-23

Conversation

@sydneyrenee
Copy link
Copy Markdown
Member

Potential fix for https://github.com/SolaceHarmony/emberharmony/security/code-scanning/23

Add an explicit permissions block at the workflow root so all jobs (including zed) inherit least-privilege defaults. For this workflow, the best non-breaking baseline is:

  • contents: read

This supports checkout/read operations while preventing unnecessary write scopes for GITHUB_TOKEN. Since the workflow appears to use PAT secrets for external/write actions, restricting GITHUB_TOKEN should not change intended functionality.

Change needed:

  • File: .github/workflows/sync-zed-extension.yml
  • Region: after the on: triggers block and before jobs:
  • No imports, methods, or extra definitions are needed (YAML config only).

Suggested fixes powered by Copilot Autofix. Review carefully before merging.

…in permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@sydneyrenee sydneyrenee marked this pull request as ready for review May 4, 2026 12:18
@sydneyrenee sydneyrenee merged commit a640c8a into dev May 4, 2026
8 checks passed
@sydneyrenee sydneyrenee deleted the alert-autofix-23 branch May 4, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant