fix: satisfy the nightly license audit - #3592
Conversation
The nightly audit fails on two licenses that arrived when the moq-net fuzz harness joined the workspace (#3543): - libfuzzer-sys 0.4.13 is (MIT OR Apache-2.0) AND NCSA; the NCSA term comes from the vendored LLVM libFuzzer harness and is OSI approved. Allow it. - moq-net-fuzz has no license field, so cargo-deny reads it as unlicensed. Declare the same MIT OR Apache-2.0 as every other workspace crate.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. WalkthroughThe fuzz crate manifest now declares Merge Risk: ⚪ Minimal · up to This change records the fuzz crate’s license and permits the NCSA license used by its vendored fuzzing dependency, without changing runtime or wire behavior. It is ready to merge. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The nightly (34473996355, also 34348703531) fails
just rs auditwith two license errors. Both arrived when the moq-net fuzz harness joined the workspace in #3543; the audit is nightly-only by design, so no PR gate caught it.libfuzzer-sys 0.4.13is(MIT OR Apache-2.0) AND NCSA. The NCSA term covers the vendored LLVM libFuzzer harness and is OSI approved / FSF Free, so allow it indeny.toml.moq-net-fuzzhas no license field, so cargo-deny treats it as unlicensed. DeclareMIT OR Apache-2.0, matching every other workspace crate.Verified with
nix develop --command just rs auditlocally. No API or wire impact.