Skip to content

Conversation

@justin-layerv
Copy link
Contributor

Summary

Add explicit permissions block to the ubuntu-build.yml workflow.

Problem

The workflow was missing explicit permissions configuration. Without this, workflows use default token permissions which may be broader than necessary.

Changes

File Change
.github/workflows/ubuntu-build.yml Add permissions: contents: read

This follows GitHub Actions security best practices by applying the principle of least privilege.

Test plan

  • Verify workflow still runs successfully on push/PR

🤖 Generated with Claude Code

@justin-layerv justin-layerv self-assigned this Jan 3, 2026
@justin-layerv justin-layerv requested a review from windcbf January 3, 2026 17:39
windcbf
windcbf previously approved these changes Jan 3, 2026
craftleon
craftleon previously approved these changes Jan 6, 2026
@craftleon craftleon force-pushed the fix/workflow-permissions branch from 8b578ae to 969791a Compare January 6, 2026 08:14
@craftleon
Copy link
Contributor

Merging is blocked due to commit missing verified signatures.

@justin-layerv justin-layerv force-pushed the fix/workflow-permissions branch from 088676a to 22fefab Compare January 6, 2026 22:37
@craftleon craftleon merged commit 40901c8 into OpenNHP:main Jan 7, 2026
6 checks passed
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.

3 participants