Skip to content

chore(ci): security hardening - #1

Merged
iota9star merged 1 commit into
mainfrom
chore/security-hardening
Aug 13, 2026
Merged

chore(ci): security hardening#1
iota9star merged 1 commit into
mainfrom
chore/security-hardening

Conversation

@iota9star

Copy link
Copy Markdown
Member

Resolves zizmor CI/CD security findings for .github/workflows/test.yml.

Closes fluttercandies/security-scanner#36

Changes:

  • unpinned-uses (High): Pinned actions/checkout, dart-lang/setup-dart, codecov/codecov-action to commit SHAs with tag comments.
  • artipacked (Medium): Added persist-credentials: false to actions/checkout.
  • excessive-permissions (Medium): Added top-level permissions: contents: read.

Copilot AI lite review requested due to automatic review settings August 13, 2026 14:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR hardens the Dart CI workflow in .github/workflows/test.yml to address zizmor-reported CI/CD security findings by minimizing default GITHUB_TOKEN permissions and removing mutable action references.

Changes:

  • Added explicit workflow-level permissions: contents: read to avoid excessive default token permissions.
  • Pinned actions/checkout, dart-lang/setup-dart, and codecov/codecov-action to immutable commit SHAs (with tag comments for traceability).
  • Disabled persisted Git credentials during checkout via persist-credentials: false.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@iota9star
iota9star merged commit 853f4c0 into main Aug 13, 2026
3 checks passed
@iota9star
iota9star deleted the chore/security-hardening branch August 13, 2026 14:21
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.

[CI/CD Security] fluttercandies/f_limit - Issue Report

2 participants