-
Notifications
You must be signed in to change notification settings - Fork 0
Integrate TrustSignal Verify Artifact in workflow #45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| name: .github/workflows/main.yml | ||
|
|
||
| on: | ||
| workflow_dispatch: | ||
| push: | ||
| branches: ["master"] | ||
|
|
||
| jobs: | ||
| verify-artifact: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
| - name: Echo placeholder | ||
| run: echo "Placeholder TrustSignal verify artifact task" | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
The Useful? React with 👍 / 👎. |
||
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Copilot Autofix
AI 3 days ago
In general, the fix is to add an explicit
permissionsblock that grants only the minimum required scopes forGITHUB_TOKEN. For this workflow, the job only checks out the repository and runs a shell command; it does not need write access, socontents: readis sufficient.The best fix without changing existing functionality is to add a
permissionssection scoped to theverify-artifactjob. This keeps the change minimal and local. At.github/workflows/main.yml, underjobs: verify-artifact:, add:indented correctly so it is a sibling of
runs-on. No imports or additional definitions are needed; this is standard GitHub Actions YAML.Concretely:
.github/workflows/main.yml.verify-artifactjob (lines 9–16).permissionsblock betweenruns-on: ubuntu-latestandsteps:(or equivalently just afterruns-on), at two spaces of indentation underverify-artifact:.