Skip to content

osv: skip ECOSYSTEM ranges for Go and npm#1908

Merged
jvdm merged 1 commit into
quay:mainfrom
jvdm:jvdm/osv-skip-ecosystem-go-npm
Jun 9, 2026
Merged

osv: skip ECOSYSTEM ranges for Go and npm#1908
jvdm merged 1 commit into
quay:mainfrom
jvdm:jvdm/osv-skip-ecosystem-go-npm

Conversation

@jvdm

@jvdm jvdm commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

The GitHub Advisory Database published advisories for the npm package nocodb using ECOSYSTEM range type instead of SEMVER (e.g., GHSA-4w6r-5c2j-qf5f). This happens because nocodb switched to calendar-based versioning (2026.04.1), which is not valid SemVer 2.0 — the advisory database chose ECOSYSTEM accordingly.

The parser currently returns a hard error for ECOSYSTEM ranges on Go and npm ecosystems, added as a defensive check in a607a05 and 87be024. This error propagates through Parse, aborting the entire ecosystem update.

This change skips the range with a warning instead, using the same pattern as GIT ranges. The advisory is recorded via stats.Ignored.

@jvdm jvdm requested a review from a team as a code owner June 9, 2026 01:15
@jvdm jvdm requested review from crozzy and hdonnay and removed request for a team June 9, 2026 01:15

@dcaravel dcaravel 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.

LGTM

The GitHub Advisory Database published advisories for the npm package
nocodb using ECOSYSTEM range type instead of SEMVER (e.g.,
GHSA-4w6r-5c2j-qf5f). This happens because nocodb switched to
calendar-based versioning (2026.04.1), which is not valid SemVer 2.0 —
the advisory database chose ECOSYSTEM accordingly.

The parser currently returns a hard error for ECOSYSTEM ranges on Go
and npm ecosystems, added as a defensive check in a607a05 and
87be024. This error propagates through Parse, aborting the entire
ecosystem update.

This change skips the range with a warning instead, using the same
pattern as GIT ranges. The advisory is recorded via stats.Ignored.

Signed-off-by: J. Victor Martins <jvdm@sdf.org>
@jvdm jvdm force-pushed the jvdm/osv-skip-ecosystem-go-npm branch from bc0c6fa to 78ab95f Compare June 9, 2026 22:45
@jvdm jvdm merged commit c99d652 into quay:main Jun 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants