feat(ci): add openssl-secjson - #206
Merged
Merged
Conversation
Wire the new vuls-data-update fetch target into the raw pipeline: fetch, archive, backup, gc and restore. The extract matrices get a commented-out entry only, since no extractor exists for this source yet. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
4 tasks
Contributor
There was a problem hiding this comment.
🟢 Ready to approve
The new target is added consistently (and alphabetically) across the intended raw-pipeline workflow lists, with extract entries correctly left commented out to match the stated fetch-only approach.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
Wires the new openssl-secjson source into the raw-data GitHub Actions pipeline by adding it to the relevant workflow target lists/matrices, while keeping extraction entries commented out (fetch-only) until an extractor exists.
Changes:
- Added
openssl-secjsontofetch-main.ymlworkflow_dispatch choices and to thefetch-all.ymlmatrix. - Added
vuls-data-raw-openssl-secjsonto raw repo target lists used by archive/backup/restore/gc workflows (includinggc-raw.ymlpack settings). - Recorded
openssl-secjsonas a commented-out entry inextract-main.yml/extract-all.ymlto reflect “fetch-only” status.
File summaries
| File | Description |
|---|---|
| .github/workflows/fetch-main.yml | Adds openssl-secjson to dispatch target choices. |
| .github/workflows/fetch-all.yml | Adds openssl-secjson to the fetch matrix to keep it in sync with dispatch options. |
| .github/workflows/archive.yml | Adds vuls-data-raw-openssl-secjson to raw archive targets. |
| .github/workflows/archive-raw.yml | Adds openssl-secjson to the raw-archive source list. |
| .github/workflows/backup-daily.yml | Adds vuls-data-raw-openssl-secjson to daily raw backup targets. |
| .github/workflows/backup-weekly.yml | Adds vuls-data-raw-openssl-secjson to weekly raw backup targets. |
| .github/workflows/backup-monthly.yml | Adds vuls-data-raw-openssl-secjson to monthly raw backup targets. |
| .github/workflows/restore-all.yml | Adds vuls-data-raw-openssl-secjson to raw restore targets. |
| .github/workflows/gc.yml | Adds vuls-data-raw-openssl-secjson to raw GC targets. |
| .github/workflows/gc-raw.yml | Adds vuls-data-raw-openssl-secjson to the GC-raw matrix with pack settings. |
| .github/workflows/extract-main.yml | Adds a commented-out openssl-secjson entry to reflect fetch-only status. |
| .github/workflows/extract-all.yml | Adds commented-out openssl-secjson entries in both extract matrices. |
Review details
- Files reviewed: 12/12 changed files
- Comments generated: 0
- Review effort level: Low
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
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.
What
Wire the new
openssl-secjsonfetch target (MaineK00n/vuls-data-update#900) into the raw data pipeline.Added
openssl-secjson/vuls-data-raw-openssl-secjsonto:fetch-main.ymlworkflow_dispatchtarget choicesfetch-all.ymlfetch-mainmatrixarchive.yml,archive-raw.ymlbackup-daily.yml,backup-weekly.yml,backup-monthly.ymlgc.yml,gc-raw.ymlpack-threads: 2,pack-windowMemory: 4g)restore-all.ymlextract-main.yml,extract-all.ymlWhy
OpenSSL publishes advisories as MITRE CVE Record Format JSON at https://openssl-library.org/news/secjson/. The new fetcher needs no special handling (no secrets, no args, no restore step), so the generic
fetch-mainpath covers it.How
Entries are placed alphabetically between
openeuler-cvrfandoracle-linuxin every list.The
extract-*matrices get a commented-out entry rather than a live one, since no extractor exists for this source yet — matching how other fetch-only sources such asmitre-emb3dandnetbsdare recorded.db-main.mk/db-nightly.mkand the diff-guard thresholds are untouched for the same reason.Testing
.github/workflows/*.ymlparse (yaml.safe_load)fetch-main.ymldispatch choices and thefetch-all.ymlmatrix stay in sync at 135 entries each, both containingopenssl-secjsonmitre-emb3d,netbsd) so no raw-pipeline list is missedFetch Mainrun for this target will be the real verification, and it requires the upstream fetcher PR to be merged tonightlyfirstChecklist
Fetch Maindispatched foropenssl-secjsonafter feat(fetch/openssl/secjson): add OpenSSL security advisory JSON fetcher MaineK00n/vuls-data-update#900 merges🤖 Generated with Claude Code