Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add a command to synchronize advisory data from osv.dev/GHSA #656

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

Merged
merged 6 commits into from
Feb 10, 2024

Conversation

amousset
Copy link
Member

@amousset amousset commented Aug 21, 2022

Result in rustsec/advisory-db#1693.

Refs: #644

This PR implements:

  • Alias id update from OSV advisories
  • Listing OSV advisories we should consider importing into RustSec

The end goal is to be able to automatically open pull requests with the advisories updates and drafts from missing ones. For now I plan to run it regularly manually until we're confident with the result.

@@ -1,5 +1,7 @@
//! The `cargo audit` subcommand

#![allow(unused_qualifications)]
Copy link
Member Author

Choose a reason for hiding this comment

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

Rust 1.70.0 produces:

warning: unnecessary qualification
  --> cargo-audit/src/commands/audit.rs:38:12
   |
38 |     #[clap(subcommand)]
   |            ^^^^^^^^^^
   |
note: the lint level is defined here
  --> cargo-audit/src/lib.rs:21:56
   |
21 | #![warn(missing_docs, rust_2018_idioms, trivial_casts, unused_qualifications)]
   |                                                        ^^^^^^^^^^^^^^^^^^^^^

I did not investigate it further for now.

Copy link
Member

Choose a reason for hiding this comment

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

I really need to upgrade clap (and abscissa)

@Shnatsel
Copy link
Member

@amousset any reason not to merge this and start running it on CI?

It would be great to have this up and running!

@amousset
Copy link
Member Author

It's been running correctly since a long time, I think it makes sense to start automating this indeed!

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

Successfully merging this pull request may close these issues.

3 participants