Working with cargo vet
Introduction
cargo vet is a tool to help ensure that third-party Rust dependencies have been audited by a trusted entity.
It matches all dependencies against a set of audits conducted by the authors of the project or entities they trust.
To learn more, visit mozilla/cargo-vet
This repository contains platform crates (platform/dev-imxrt, platform/dev-mcxa, platform/dev-npcx, platform/dev-qemu), each with its own Cargo.lock and its own supply-chain/ store. Run all cargo vet commands from the relevant platform directory.
Adding a new dependency
When updating or adding a new dependency, we need to ensure it’s audited before being merged into main.
For our repositories, we have designated experts who are responsible for vetting any new dependencies being added to their repository.
It is the shared responsibility of the developer creating the PR and the auditors to conduct a successful audit.
Follow the process below to ensure compliance:
For Developers
- Respond to
cargo vetfailures:
- If your PR fails the
cargo vetstep, the cargo-vet workflow will add a comment to the PR with a template questionnaire - Copy the questionnaire, fill it out and paste it as a new comment on the PR. This greatly helps the auditors get some context of the changes requiring the new dependencies
- Engage with auditors:
- Respond to any questions that the auditors might have regarding the need of any new dependencies
- Rebase and verify:
- At their discretion, auditors will check in their audits into either rust-crate-audits or into the same repository
- Once the new audits have been merged, rebase your branch on main and verify it passes
cargo vetgit fetch upstream git rebase upstream/main cargo vet
- Update PR:
- If the audits were checked into rust-crate-audits, they will show up in imports.lock on running
cargo vet. In this case add the updated imports.lock to your PR - If the audits were checked into the same repository, they will be present in audits.toml after rebase and you can simply force push to your PR after rebase
git push -f
- Check PR status:
- The existing PR comment from the previous failure will be updated with a success message once the check passes
For Auditors
- Review the questionnaire:
- Check the filled questionnaire on the PR once the developer responds to the
cargo vetfailure - Respond to the developer comment in case more information is needed
- Audit new dependencies:
- Inspect the
cargo vetfailures using your preferred method- Use gh pr checkout to checkout the PR and run
cargo vet --locked - Use GitHub Pull Requests for Visual Studio Code to checkout the PR and run
cargo vet --locked - For more suggestions: Checking out pull requests locally
- Use gh pr checkout to checkout the PR and run
- Follow
cargo vetrecommendations:
- Follow the recommendations of the
cargo vetcommand output, eithercargo vet difffor version update orcargo vet inspectfor new dependencies
- Record audits:
- Use
cargo vet certifyto add new audits to audits.toml - Verify all dependencies pass using
cargo vet
- Decide audit location:
- Shared audits: New audits should ideally be shared across ODP repositories to reduce the overhead of multiple audits for the same dependencies. To facilitate this, it’s recommended to cut and paste the new audits and submit as a separate PR to the audits.toml in rust-crate-audits
- If due to business reasons, the audits are not to be shared across repositories, copy the updated audits.toml to a new branch off main in the same repository and submit the PR to update the audits
- Communicate successful audit:
- Communicate to the PR developer via a PR comment so they can update the PR and get
cargo vetto pass
Audit criteria
cargo vet comes pre-equipped with two built-in criteria but supports adding new criteria to suit our needs.
As defined here, the default criteria are:
-
safe-to-run
This crate can be compiled, run, and tested on a local workstation or in controlled automation without surprising consequences, such as:- Reading or writing data from sensitive or unrelated parts of the filesystem.
- Installing software or reconfiguring the device.
- Connecting to untrusted network endpoints.
- Misuse of system resources (e.g. cryptocurrency mining).
-
safe-to-deploy
This crate will not introduce a serious security vulnerability to production software exposed to untrusted input.Auditors are not required to perform a full logic review of the entire crate. Rather, they must review enough to fully reason about the behavior of all unsafe blocks and usage of powerful imports. For any reasonable usage of the crate in real-world software, an attacker must not be able to manipulate the runtime behavior of these sections in an exploitable or surprising way.
Ideally, all unsafe code is fully sound, and ambient capabilities (e.g. filesystem access) are hardened against manipulation and consistent with the advertised behavior of the crate. However, some discretion is permitted. In such cases, the nature of the discretion should be recorded in the
notesfield of the audit record.For crates which generate deployed code (e.g. build dependencies or procedural macros), reasonable usage of the crate should output code which meets the above criteria.
Note:
safe-to-deployimpliessafe-to-run
Conducting an audit
When performing an audit for a new or updated dependency, auditors may consider the following criteria to ensure the safety, reliability, and suitability of the crate for use in our projects:
-
Security:
- Review the crate for known vulnerabilities or security advisories.
- Check for unsafe code usage and ensure it is justified and well-documented.
- Evaluate the crate’s history of security issues and responsiveness to reported problems.
-
Maintenance and Activity:
- Assess the frequency of updates and the responsiveness of maintainers to issues and pull requests.
- Prefer crates that are actively maintained and have a healthy contributor base.
-
License Compliance:
- Verify that the crate’s license is compatible with our project’s licensing requirements.
-
Community Trust and Adoption:
- Consider the crate’s adoption in the wider Rust ecosystem.
- Prefer crates that are widely used and trusted by the community.
-
Functionality and Suitability:
- Confirm that the crate provides the required functionality without unnecessary features or bloat.
- Evaluate whether the crate’s API is stable and unlikely to introduce breaking changes unexpectedly.
-
Audit Trail:
- Record the audit decision, including any concerns, mitigations, or recommendations for future updates.
- If exemptions are granted, document the justification and any follow-up actions required.
Tips for using cargo vet:
-
Update imports.lock:
- Import trusted third party audits to reduce the number of new audits to be performed. Running
cargo vetwithout--lockedfetches new imports and updates imports.lock with any audits that are helpful for our project.
- Import trusted third party audits to reduce the number of new audits to be performed. Running
-
Add exemptions:
- If an audit cannot be performed for some dependency due to time sensitivity or business justified reasons, use
cargo vet add-exemption <PACKAGE> <VERSION>to add the dependency to exemptions in config.toml - To add all remaining audits to exemptions at once, use
cargo vet regenerate exemptions
- If an audit cannot be performed for some dependency due to time sensitivity or business justified reasons, use
-
Prune unnecessary entries:
- Remove unnecessary exemptions and imports using
cargo vet prune
- Remove unnecessary exemptions and imports using