Skip to content

Limit read permission to license-check.yml workflow#1312

Merged
wuchong merged 2 commits intomainfrom
alert-autofix-1
Jul 12, 2025
Merged

Limit read permission to license-check.yml workflow#1312
wuchong merged 2 commits intomainfrom
alert-autofix-1

Conversation

@wuchong
Copy link
Copy Markdown
Member

@wuchong wuchong commented Jul 12, 2025

Potential fix for https://github.com/apache/fluss/security/code-scanning/1

To fix the issue, add a permissions block to the workflow to explicitly restrict the permissions of the GITHUB_TOKEN to the minimum required. Since the workflow involves checking out the repository code and running commands, it likely only needs contents: read. No write permissions appear necessary for this workflow.

The permissions block should be added at the root level of the workflow (directly under the name key) to apply to all jobs. This ensures consistent and minimal permissions across the entire workflow.


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>
@wuchong wuchong changed the title Potential fix for code scanning alert no. 1: Workflow does not contain permissions Limit read permission to license-check.yml workflow Jul 12, 2025
@wuchong wuchong marked this pull request as ready for review July 12, 2025 08:07
@wuchong wuchong merged commit b99e3b6 into main Jul 12, 2025
4 checks passed
@wuchong wuchong deleted the alert-autofix-1 branch July 12, 2025 09:22
polyzos pushed a commit to polyzos/fluss that referenced this pull request Aug 30, 2025
polyzos pushed a commit to Alibaba-HZY/fluss that referenced this pull request Aug 31, 2025
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