spinframework / spinframework/spin
Address `cargo audit` issues
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 6.5k
- Forks
- 310
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 24
Description
#3320 addressed some of the issues raised by cargo audit: the ones that do not require code changes. The remainder will require one or more of the following:
- upgrading to newer, API-breaking dependency versions, where available, and fixing any compilation issues
- submitting upstream patches to dependencies which are maintained but have
cargo auditissues - finding replacements for any no-longer-maintained dependencies
The output of cargo audit is quite verbose (even with the -q option), so I won't paste it here. To see the full list, run cargo install cargo-audit --locked --features=fix && cargo audit at the root of the Spin repo. The main culprit as of this writing is watchexec (used by spin-watch) , which transitively depends on versions of gix-* crates with various vulnerabilities. Outside of that, the remaining issues are due to transitive deps with no known vulnerabilities but which are no longer maintained (but might be maintained under a different crate name in some cases).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
From the root of the Spin repository, install cargo-audit with cargo install cargo-audit --locked --features=fix and run cargo audit to review the remaining findings. Start with watchexec, used by spin-watch, and its transitive gix-* dependencies; done means resolving the reported vulnerabilities and unmaintained dependencies, including any required compilation fixes or upstream work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- security, tooling
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100