Skip to content

ci: Align rust-gems RustSec audit workflow - #177

Merged
gorzell merged 4 commits into
mainfrom
gorzell/fictional-broccoli
Nov 9, 2026
Merged

gorzell merged 4 commits into
mainfrom
gorzell/fictional-broccoli

Conversation

@gorzell

@gorzell gorzell commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Adds a RustSec audit workflow for rust-gems. This also checks in a root Cargo.lock, updates .gitignore for lockfile auditing, and passes --locked in the Makefile build target.

@gorzell
gorzell requested a review from a team as a code owner November 9, 2026 12:19
Copilot AI dynamic review requested due to automatic review settings November 9, 2026 12:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

CI does not enforce lockfile consistency, so builds can use a different dependency graph than both audit jobs scan, potentially missing vulnerabilities in built dependencies.

1 open issue
What changed in this PR

Aligns rust-gems’ RustSec auditing with the Blackbird workflow approach and adds a tracked dependency lockfile for workspace audits.

Changes:

  • Adds push-triggered and manual audits, plus nightly audits with issue-reporting permissions.
  • Commits the root Cargo.lock as the audit input.
  • Allows tracking the root lockfile while keeping nested lockfiles ignored.
File Description
Cargo.lock Records workspace dependency versions for auditing.
.gitignore Exempts the root lockfile from ignore rules.
.github/​workflows/​security-audit.yml Adds event-driven and scheduled RustSec audit jobs.

Review effort: Balanced (auto)

Comment thread .github/workflows/security-audit.yml Resolved

@tclem tclem left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Source review found no new actionable issues. Local validation was blocked.

  Generated via Copilot (GPT-6.1 Sol) on behalf of @tclem

@gorzell
gorzell merged commit 5a618d1 into main Nov 9, 2026
9 checks passed
@gorzell
gorzell deleted the gorzell/fictional-broccoli branch November 9, 2026 15:59
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.

4 participants