Skip to content

Add contents read permission to snapshot workflow#33

Merged
bernardladenthin merged 1 commit into
masterfrom
claude/add-workflow-permissions-zyUcf
May 7, 2026
Merged

Add contents read permission to snapshot workflow#33
bernardladenthin merged 1 commit into
masterfrom
claude/add-workflow-permissions-zyUcf

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

Added explicit contents: read permission to the snapshot workflow's build job to follow GitHub Actions security best practices.

Key Changes

  • Added permissions block with contents: read to the build job in .github/workflows/snapshot.yml

Implementation Details

This change implements the principle of least privilege by explicitly declaring the minimum required permissions for the workflow job. The contents: read permission allows the job to read repository contents (needed for the actions/checkout@v4 step) while restricting other potentially unnecessary permissions by default.

https://claude.ai/code/session_018GqGrJWwE6Fhw9UPy6ViEv

Add permissions: contents: read to the build job to enforce least-privilege
on the GITHUB_TOKEN, as suggested by CodeQL.
@codecov

codecov Bot commented May 7, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
267 1 266 0
View the top 1 failed test(s) by shortest run time
mutation survivors — boundary conditions and arithmetic::trim_closeCalledDuringTrim_handlesGracefully
Stack Traces | 0.014s run time
Multiple Failures (1 failure)
	java.lang.AssertionError: Trim should not throw exception
Expected: is null
     but: was <java.io.IOException: Stream closed.>

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@bernardladenthin bernardladenthin merged commit ae6f379 into master May 7, 2026
11 of 14 checks passed
@bernardladenthin bernardladenthin deleted the claude/add-workflow-permissions-zyUcf branch May 7, 2026 16:54
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