Skip to content

ci: run osv-detector#73

Merged
G-Rath merged 1 commit into
mainfrom
ci/run-osv-detector
Jun 26, 2025
Merged

ci: run osv-detector#73
G-Rath merged 1 commit into
mainfrom
ci/run-osv-detector

Conversation

@G-Rath

@G-Rath G-Rath commented Jun 15, 2025

Copy link
Copy Markdown
Collaborator

This will help ensure that our database is healthy since our config has this database loaded as a local file database

Comment thread .github/workflows/ci.yml Fixed

@jonholdsworth jonholdsworth left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Forgive me, but I don't understand what this is doing?

@G-Rath

G-Rath commented Jun 15, 2025

Copy link
Copy Markdown
Collaborator Author

@jonholdsworth this runs osv-detector on the codebase to check for known vulnerabilities like we do for our other codebases - a useful difference here though is that this codebase is also a source of OSV advisories which osv-detector is configured to use when running at the root of the codebase, so in addition to its standard auditing this also serves as an integration test as any invalid advisories will be rejected by osv-detector when it loads the database

@jonholdsworth jonholdsworth self-requested a review June 15, 2025 23:32

@jonholdsworth jonholdsworth left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Aha, a meta-check! 🙂

@G-Rath G-Rath force-pushed the ci/run-osv-detector branch 2 times, most recently from fa95e01 to 7adc3ac Compare June 17, 2025 20:14
@G-Rath G-Rath force-pushed the ci/run-osv-detector branch from 7adc3ac to 6ad18df Compare June 24, 2025 00:50
@G-Rath G-Rath force-pushed the ci/run-osv-detector branch from 6ad18df to a911c98 Compare June 26, 2025 03:03
@G-Rath G-Rath merged commit c4f6516 into main Jun 26, 2025
9 checks passed
@G-Rath G-Rath deleted the ci/run-osv-detector branch June 26, 2025 04:19
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.

4 participants