Skip to content

ci: Try warden - #317

Closed
lcian wants to merge 2 commits into
mainfrom
lcian/warden
Closed

lcian wants to merge 2 commits into
mainfrom
lcian/warden

Conversation

@lcian

@lcian lcian commented Feb 18, 2026

Copy link
Copy Markdown
Member

No description provided.

Comment on lines +21 to +23
- uses: actions/checkout@v4
- uses: getsentry/warden@v0
with:

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟠 [EZE-BEU] GitHub Actions not pinned to commit hashes (medium confidence)

Actions use version tags (@v4, @v0) instead of commit hashes. Version tags can be moved to point to different commits, potentially allowing supply chain attacks if the action repository is compromised.

Suggested fix: Pin actions to specific commit hashes for better supply chain security. Look up the current commit for each version tag and use that hash.

Suggested change
- uses: actions/checkout@v4
- uses: getsentry/warden@v0
with:
- uses: actions/checkout@<COMMIT_HASH> # v4
- uses: getsentry/warden@<COMMIT_HASH> # v0

Identified by Warden via security-review · medium, medium confidence

- Switch from default GITHUB_TOKEN to dedicated GitHub App token
- Narrow permissions to contents: read
- Add find-bugs, security-review, code-review, and code-simplifier skills
- Remove WARDEN_SENTRY_DSN
@lcian lcian closed this Feb 20, 2026
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