Skip to content

fix: remove vulnerable regexp2 v1 and upgrade dependencies - #339

Merged
daveshanley merged 2 commits into
mainfrom
codex/regexp2-v2-security
Oct 30, 2026
Merged

daveshanley merged 2 commits into
mainfrom
codex/regexp2-v2-security

Conversation

@daveshanley

Copy link
Copy Markdown
Member

Removes regexp2 v1 from the validator’s compiled packages to address CVE-2026-77299, reported in daveshanley/vacuum#1023. Arazzo validation and the CLI now use regexp2 v2 with its default backtracking bound.

  • Preserve CLI options, including debug and the legacy no-op compiled alias.
  • Align the Arazzo fast matcher with ECMAScript Unicode line-separator behavior and add bounded-repeat regression tests.
  • Upgrade all out-of-date dependencies declared in go.mod: regexp2/v2 v2.12.0, libopenapi v0.41.8, jsonpath v0.8.5, jsonparser v1.6.1, and x/net v0.62.0.

Validation: full validator test suite passed with GOWORK=off; independent correctness and de-slop reviews found no remaining material issues. Focused Arazzo and CLI race tests passed; the rebuilt validator binary contains regexp2 v2.12.0 and no regexp2 v1.

The v1 checksum retained through a pb33f/jsonschema example-test dependency is not part of the compiled package closure. A validator release and a subsequent Vacuum dependency update are needed to ship this fix to Vacuum users.

@codecov

codecov Bot commented Oct 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.74%. Comparing base (d7a43ec) to head (701e0c4).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #339      +/-   ##
==========================================
+ Coverage   98.66%   98.74%   +0.08%     
==========================================
  Files          94       94              
  Lines       12310    12310              
==========================================
+ Hits        12146    12156      +10     
+ Misses        139      129      -10     
  Partials       25       25              
Flag Coverage Δ
unittests 98.74% <100.00%> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@daveshanley
daveshanley marked this pull request as ready for review October 30, 2026 19:25
@daveshanley
daveshanley merged commit 754a5b7 into main Oct 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant