Skip to content

Conversation

IstvanZsSzekely
Copy link
Collaborator

Potential fix for https://github.com/analogdevicesinc/testbenches/security/code-scanning/1

To fix this issue, we should add a permissions: block to the build-doc job in .github/workflows/top-level.yml, restricting the GITHUB_TOKEN to the minimum access required. Since the job only builds documentation and uploads artifacts, and does not need to push to the repository or access issues, contents: read is sufficient. This block should be added beneath runs-on: ubuntu-latest (line 9) and before the steps: block for the build-doc job. No other modification is needed. The deploy job already correctly sets its own permissions:.

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

…n permissions

To fix this issue, we should add a permissions: block to the build-doc job in .github/workflows/top-level.yml, restricting the GITHUB_TOKEN to the minimum access required. Since the job only builds documentation and uploads artifacts, and does not need to push to the repository or access issues, contents: read is sufficient. This block should be added beneath runs-on: ubuntu-latest (line 9) and before the steps: block for the build-doc job. No other modification is needed. The deploy job already correctly sets its own permissions:.

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@IstvanZsSzekely IstvanZsSzekely self-assigned this Oct 17, 2025
@IstvanZsSzekely IstvanZsSzekely added the help wanted Extra attention is needed label Oct 17, 2025
github-actions bot added a commit that referenced this pull request Oct 17, 2025
@IstvanZsSzekely IstvanZsSzekely marked this pull request as ready for review October 17, 2025 14:08
@IstvanZsSzekely IstvanZsSzekely requested a review from a team as a code owner October 17, 2025 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant