Add OSV Scanner config for test fixtures - #1862
Open
el kampu (elkampu) wants to merge 1 commit into
Open
Conversation
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The changes only add directory-scoped OSV Scanner configuration for non-shipping test fixtures/mocks and do not alter product code or detector behavior.
Pull request overview
Adds directory-local osv-scanner.toml configuration files alongside existing test/fixture content so OpenSSF Scorecard and OSV Scanner don’t report known-vulnerable (but intentionally outdated and non-shipping) verification resources and detector-report mocks as repository vulnerabilities.
Changes:
- Add per-directory OSV Scanner ignore lists for advisories triggered by intentionally outdated verification test fixtures.
- Add OSV Scanner ignore lists for two detector test mock directories that appear in repository-wide scans.
- Document the suppression rationale consistently via per-advisory
reasonentries (no package-wide/repo-wide suppressions).
File summaries
| File | Description |
|---|---|
| test/Microsoft.ComponentDetection.VerificationTests/resources/yarn/v2/osv-scanner.toml | Local OSV ignore list for Yarn v2 fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/yarn/v1/osv-scanner.toml | Local OSV ignore list for Yarn v1 fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/uv/osv-scanner.toml | Local OSV ignore list for uv fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/rust/workspaces/osv-scanner.toml | Local OSV ignore list for Rust workspace fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/rust/standard/osv-scanner.toml | Local OSV ignore list for standard Rust fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/poetry/osv-scanner.toml | Local OSV ignore list for Poetry fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pnpm/v6/osv-scanner.toml | Local OSV ignore list for pnpm v6 fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pnpm/v5/osv-scanner.toml | Local OSV ignore list for pnpm v5 fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/roots/osv-scanner.toml | Local OSV ignore list for pip “roots” fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/pre-generated/simple/osv-scanner.toml | Local OSV ignore list for pre-generated pip “simple” fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/pre-generated/multiple/osv-scanner.toml | Local OSV ignore list for pre-generated pip “multiple” fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/pre-generated/invalid/osv-scanner.toml | Local OSV ignore list for pre-generated pip “invalid” fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/parallel/parallel-test-5/osv-scanner.toml | Local OSV ignore list for pip parallel test fixture 5. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/parallel/parallel-test-4/osv-scanner.toml | Local OSV ignore list for pip parallel test fixture 4. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/parallel/parallel-test-3/osv-scanner.toml | Local OSV ignore list for pip parallel test fixture 3. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/parallel/parallel-test-2/osv-scanner.toml | Local OSV ignore list for pip parallel test fixture 2. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/parallel/parallel-test-1/osv-scanner.toml | Local OSV ignore list for pip parallel test fixture 1. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/osv-scanner.toml | Local OSV ignore list for pip fixtures at resources/pip. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/index-removal/osv-scanner.toml | Local OSV ignore list for pip index-removal fixture. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/pip/fallback/osv-scanner.toml | Local OSV ignore list for pip fallback fixture. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/npm/shrinkwrap/osv-scanner.toml | Local OSV ignore list for npm shrinkwrap fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/npm/lockfile3/osv-scanner.toml | Local OSV ignore list for npm lockfile v3 fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/npm/lockfile/osv-scanner.toml | Local OSV ignore list for npm lockfile fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/gradle/osv-scanner.toml | Local OSV ignore list for Gradle fixtures. |
| test/Microsoft.ComponentDetection.VerificationTests/resources/go/osv-scanner.toml | Local OSV ignore list for Go fixtures. |
| test/Microsoft.ComponentDetection.Detectors.Tests/Mocks/InvalidJsonReport/osv-scanner.toml | Local OSV ignore list for “InvalidJsonReport” mock fixture directory. |
| test/Microsoft.ComponentDetection.Detectors.Tests/Mocks/EmptyReport/osv-scanner.toml | Local OSV ignore list for “EmptyReport” mock fixture directory. |
Review details
- Files reviewed: 27/27 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
osv-scanner.tomlbeside each test fixture directory currently reported by the Scorecard vulnerability check.OSV Scanner configuration is directory-local and does not propagate to child directories, so repeated fixture layouts need colocated files. Each ignore records that the dependency is intentionally outdated test data that is not built or shipped.
Closes #1073
Validation
Vulnerabilities: 10/10,0 existing vulnerabilities detectedEnforceCodeStyleInBuild=falseDockerService_CanPingDockerAsyncbecause Docker is unavailable on the test hostgit diff --checkThe unmodified build command also reports pre-existing IDE suggestions as errors in unchanged C# files on this Windows checkout; upstream CI is green on the same base commit.