Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: replace cargo audit with cargo deny #1597

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LesnyRumcajs
Copy link
Contributor

Replaced cargo audit with a more comprehensive cargo deny. Note that the currently used Cargo Audit Action is no longer maintained and repository archived.

The included deny.toml (mostly generated with cargo deny init) includes advisory exceptions on dependencies, which should be eventually tackled. I can create corresponding issues - from a quick glimpse most have a known solution (update or replace the dep with a maintained one).

We might want to further customize the configuration file, e.g. if licensing is something we need to watch for.

#db-urls = ["https://github.com/rustsec/advisory-db"]
# A list of advisory IDs to ignore. Note that ignored advisories will still
# output a note when they are encountered.
ignore = [
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Currently suppressed advisories.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔎 Awaiting review
Development

Successfully merging this pull request may close these issues.

1 participant