Skip to content

Conversation

@SilanHe
Copy link
Contributor

@SilanHe SilanHe commented Sep 24, 2025

Potential fix for https://github.com/aws/aws-durable-execution-sdk-python-testing/security/code-scanning/1

To resolve the problem, add a permissions block to the workflow to explicitly set the minimal required GitHub token permissions. Because the workflow does not perform any write operations (to contents, issues, pull-requests, etc.), it is sufficient to give read-only access to repository contents. Place the permissions block at the top level of the workflow, directly below the name field (affecting all jobs in the workflow), as recommended by GitHub best practices.

What to do:

  • Edit .github/workflows/ci.yml.
  • Add:
    permissions:
      contents: read
    after the name property and before the on property.
  • No additional imports or variable definitions are needed.

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>
@SilanHe SilanHe marked this pull request as ready for review September 24, 2025 18:08
@SilanHe SilanHe merged commit 548e936 into main Sep 24, 2025
6 checks passed
@SilanHe SilanHe deleted the alert-autofix-1 branch September 24, 2025 18: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.

2 participants