Repository navigation
Allow overriding the SCA CVE catalog from a local file - #13285
Conversation
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. |
There was a problem hiding this comment.
More details
Source and test inspection supports the documented behavior: an unset override retains the bundled catalog, while unreadable or malformed overrides leave the database empty without enabling SCA.
🤖 Bits Code Review · Commit 89516cc
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
1724636
into
master
What Does This Do
Add the internal
_DD_APPSEC_SCA_CVE_FILEenvironment variable to load SCA advisories from a local JSON file instead of the bundled catalog. When unset, the agent uses its bundled catalog.Motivation
Allow tests to supply synthetic advisories and deployments to update the catalog without modifying the agent JAR, matching the Python override.
Additional Notes
The file uses the existing Java catalog schema and replaces the bundle without merging. An unreadable or malformed override loads no targets and does not fall back to the bundle. The override does not enable SCA and is excluded from configuration telemetry.
Validation: all 103 SCA tests and 497 Config tests pass. Java formatting and configuration-registry checks pass.