GHSA advisories never use the OSV `FIX` reference type, though ~54% link a fix commit
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 38/100
- Issue type
- Feature
- Clarity
- Needs clarification
- Activity status
- Active
- Domain
- security
Research direction
Start by reproducing the measurement against the published OSV per-ecosystem archives and inspect the example GHSA-35jh-r3h4-6jhm references. Read the repository’s advisory curation workflow to determine how reference types are assigned; done means the maintainers have decided whether and how fix commits should receive FIX labels, with the workflow or data changes defined.
Written by the indexing model from the issue text.
Description
Summary
Across every GHSA advisory published to the OSV feeds, the FIX reference type
is used zero times. Meanwhile about 54% of those advisories do link their fixing
commit, filed under WEB alongside vendor bulletins and NVD mirrors.
Consumers that want the fix commit therefore have to recognise it by URL shape
rather than by reading the reference type.
Measurement
Taken from the published OSV per-ecosystem archives at
https://storage.googleapis.com/osv-vulnerabilities/<ecosystem>/all.zip,
excluding withdrawn advisories. GHSA records only.
| Ecosystem feed | GHSA records | FIX refs |
Links a commit under another type |
|---|---|---|---|
| npm | 7,020 | 0 | 3,634 (52%) |
| Maven | 6,956 | 0 | 3,268 (47%) |
| Packagist | 6,347 | 0 | 3,464 (55%) |
| PyPI | 6,016 | 0 | 3,776 (63%) |
| Go | 4,607 | 0 | 2,646 (57%) |
| crates.io | 1,475 | 0 | 650 (44%) |
| RubyGems | 1,082 | 0 | 639 (59%) |
| NuGet | 1,075 | 0 | 511 (48%) |
| Total | 34,578 | 0 | 18,588 (53.8%) |
The field is in use elsewhere
This is not a dead part of the schema. Other databases in the same feeds
populate it:
| Source | Records | Using FIX |
|---|---|---|
GO-* (Go vulndb) |
4,362 | 2,300 (52.7%) |
PYSEC-* (PyPA) |
7,214 | 2,146 (29.7%) |
GHSA-* |
34,578 | 0 (0.0%) |
Example
GHSA-35jh-r3h4-6jhm (lodash, CVE-2021-23337) has 17 references. The fix commit
is present, typed WEB:
ADVISORY https://nvd.nist.gov/vuln/detail/CVE-2021-23337
WEB https://github.com/lodash/lodash/commit/3469357cff396a26c363f8c1b5a91dde28ba4b1c
WEB https://www.oracle.com/security-alerts/cpuoct2021.html
WEB https://snyk.io/vuln/SNYK-JS-LODASH-1040724
PACKAGE https://github.com/lodash/lodash
...
Why it matters
A consumer wanting the fix commit currently has to pattern-match commit URLs
across GitHub, GitLab and Bitbucket against every reference. That heuristic will
miss self-hosted forges and will occasionally match a link that is not the fix.
This came up while building tooling that uses fix-commit diffs to corroborate
advisory metadata. It is relevant to anything doing patch analysis, reachability
work, or backport detection.
The intent also seems to already be there. In an OpenJS discussion on affected
function naming, a maintainer noted:
For what it's worth we do tend to also provide fix commits when we can find
them which would allow someone to derive the file based location data.
That derivation would be considerably more reliable if the references were
labelled.
Question
Is there a reason FIX is not used, or is it simply not something the curation
workflow currently sets? I have no visibility into how references get typed, so
I may be missing context that makes this a non-issue.
Offer to help
If this is worth addressing and there is a useful way for me to contribute, I am
happy to. I recognise that a change spanning this many advisories is not
something to approach through the one-advisory-per-PR path, and that you likely
have better tooling for it than anything I would write.
So rather than guessing at an approach, I would rather ask: if you want this
fixed, what would actually help? I can share the measurement script, produce a
list of candidate advisories and the reference in each that looks like a fix
commit, or do something else entirely. Guidance welcome.
Reproducing the numbers
curl -O https://storage.googleapis.com/osv-vulnerabilities/npm/all.zip
mkdir feed && unzip -q all.zip -d feed
# count GHSA records, excluding withdrawn, whose references include type FIX
The same holds for every ecosystem archive. Happy to share the full script.
- Dominant language
- No language data
- Stars
- 2.5k
- Forks
- 772
- Avg merge
- 3d 18h
- Merged PRs (30d)
- 48
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from github/advisory-database
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
github/advisory-database#9255 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
github/advisory-database#9164 · 1 reaction ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
github/advisory-database#8994 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
github/advisory-database#8898 · 4 comments · 1 reaction ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
github/advisory-database#8841 ·
All issues in github/advisory-database
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
punkpeye/mcp-remote#369 ·
-
Mend: dependency security vulnerability untriaged
Difficulty 1/5 Under an hour Newbie friendliness 86/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 90/100
cisagov/vulnrichment#337 ·
-
bug DUP Reservations
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
bcgov/reserve-rec-public#896 ·