Skip to content

chore: onboard stepsecurity and apply security best practice - #48

Open
ali-kafel wants to merge 2 commits into
circlefin:masterfrom
ali-kafel:patch-1
Open

ali-kafel wants to merge 2 commits into
circlefin:masterfrom
ali-kafel:patch-1

Conversation

@ali-kafel

Copy link
Copy Markdown
Member

Summary

This pull request improves the security and reliability of the GitHub Actions workflows for end-to-end and unit tests. The main changes include hardening the runners, specifying permissions more explicitly, and pinning action versions to specific commit SHAs for better reproducibility.

Security hardening:

  • Added the step-security/harden-runner action to both .github/workflows/e2e-tests.yaml and .github/workflows/unit-tests.yaml to restrict network egress and enforce a global allowed endpoints policy. [1] [2]

Permissions improvements:

  • Explicitly set permissions at the workflow and job levels, including contents: read and id-token: write, to follow the principle of least privilege. [1] [2]

Reliability and reproducibility:

  • Updated actions/checkout and actions/setup-go to use pinned commit SHAs instead of version tags, ensuring consistent action versions across workflow runs. [1] [2]

Testing

Documentation


Requested Reviewers: @mention

stepsecurity-app Bot and others added 2 commits October 30, 2025 17:42
Signed-off-by: StepSecurity Bot <bot@stepsecurity.io>
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