Skip to content

Potential fix for code scanning alert no. 6: Workflow does not contain permissions#112

Merged
yo-han merged 1 commit into
developfrom
alert-autofix-6
Oct 17, 2025
Merged

Potential fix for code scanning alert no. 6: Workflow does not contain permissions#112
yo-han merged 1 commit into
developfrom
alert-autofix-6

Conversation

@yo-han

@yo-han yo-han commented Oct 17, 2025

Copy link
Copy Markdown
Owner

Potential fix for https://github.com/yo-han/Home-Assistant-Carelink/security/code-scanning/6

To fix the problem, add an explicit permissions key to the workflow—either at the root (to apply to all jobs unless overridden) or directly within jobs as needed. Since this workflow only checks out code and runs hassfest without creating or modifying issues, pull requests, or any repo contents, it is safe and optimal to assign contents: read—the minimal required permission. Place permissions: contents: read at the top level, right after the name and before the on: key, in .github/workflows/hassfest.yml.

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

…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@yo-han yo-han marked this pull request as ready for review October 17, 2025 07:20
@yo-han yo-han merged commit 0832f0c into develop Oct 17, 2025
2 checks passed
@yo-han yo-han deleted the alert-autofix-6 branch January 8, 2026 09:45
jnctech pushed a commit to jnctech/ha-tandem-pump that referenced this pull request Mar 9, 2026
…n permissions (yo-han#112)

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
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