Add cargo-vet and cargo-deny supply-chain security#12
Open
jerrysxie wants to merge 1 commit into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
Adds supply-chain security tooling to the crate by introducing cargo-vet configuration/audit stubs under supply-chain/ and new GitHub Actions workflows to run cargo vet on PRs and post/update a PR comment with the result.
Changes:
- Add initial
cargo-vetconfig/audits/import stubs insupply-chain/. - Add
cargo-vetCI workflow to runcargo vet --lockedon pull requests. - Add a
workflow_run-triggered workflow to comment on PRs whencargo-vetfails (and update the comment on later success).
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| supply-chain/imports.lock | Initializes cargo-vet import lock structure for external audit sources. |
| supply-chain/config.toml | Configures cargo-vet and defines upstream audit import URLs. |
| supply-chain/audits.toml | Adds a placeholder local audits file for cargo-vet. |
| .github/workflows/cargo-vet.yml | Introduces a PR CI job that installs and runs cargo-vet. |
| .github/workflows/cargo-vet-pr-comment.yml | Adds a follow-up workflow to comment/label PRs based on cargo-vet results. |
Comments suppressed due to low confidence (1)
.github/workflows/cargo-vet-pr-comment.yml:75
- The failure comment links to
supply-chain/README.md, but that file doesn’t exist in this repo (thesupply-chain/directory only containsaudits.toml,config.toml, andimports.lock). This link will be broken for contributors; either add the referenced README or update the link to an existing doc.
`cargo vet` has failed in this PR. Please run `cargo vet --locked` locally to check for new or updated unvetted dependencies.
Details about the vetting process can be found in [supply-chain/README.md](../blob/main/supply-chain/README.md)
Comment on lines
+16
to
+20
| name: vet-dependencies | ||
| runs-on: ubuntu-latest | ||
| env: | ||
| CARGO_VET_VERSION: 0.10.1 | ||
|
|
Comment on lines
+8
to
+13
| on: | ||
| workflow_run: | ||
| workflows: [cargo-vet] | ||
| types: | ||
| - completed | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds supply-chain security tooling based on the
embedded-rust-template:
supply-chain/) – dependency audit tracking with importsfrom ODP shared audits, Google, and Mozilla.
deny.toml) – license, advisory, ban, and source checks.cargo-vet.yml+ PR comment workflow, and thedenyjob incheck.yml.After merging, run
cargo vetlocally to populate exemptions for anyexisting unaudited dependencies: