Skip to content

[codex] fix path-to-regexp dependabot alert#3969

Open
szdziedzic wants to merge 2 commits into
mainfrom
szdziedzic-codex/fix-dependabot-343-path-to-regexp
Open

[codex] fix path-to-regexp dependabot alert#3969
szdziedzic wants to merge 2 commits into
mainfrom
szdziedzic-codex/fix-dependabot-343-path-to-regexp

Conversation

@szdziedzic

@szdziedzic szdziedzic commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Why

Dependabot reported alert 343 for path-to-regexp (GHSA-37ch-88jc-xwx2, CVE-2026-4867). Versions below 0.1.13 are vulnerable to ReDoS when routes contain multiple parameters in a single segment.

How

Bumped the eas-cli package's Express 4 dependency from 4.20.0 to 4.22.2, which allows the Express route parser dependency to resolve to patched path-to-regexp@0.1.13. The existing koa-router path continues to use path-to-regexp@8.3.0. Added a changelog entry for the security fix.

Test Plan

  • corepack yarn fmt
  • corepack yarn install --immutable
  • corepack yarn why path-to-regexp
  • corepack yarn lint-changelog
  • git diff --check

corepack yarn install --immutable still reports the existing peer dependency warnings for graphql, oxlint-tsgolint, eslint, and @types/node.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

✅ Thank you for adding the changelog entry!

@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.32%. Comparing base (4855745) to head (f5f37ce).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3969   +/-   ##
=======================================
  Coverage   59.32%   59.32%           
=======================================
  Files         936      936           
  Lines       41169    41169           
  Branches     8670     8670           
=======================================
  Hits        24421    24421           
  Misses      16654    16654           
  Partials       94       94           

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@szdziedzic szdziedzic requested a review from sjchmiela July 6, 2026 21:05
@szdziedzic szdziedzic marked this pull request as ready for review July 6, 2026 21:05
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Subscribed to pull request

File Patterns Mentions
**/* @douglowder

Generated by CodeMention

Warning: The preamble and epilogue options in commentConfiguration are deprecated. Use template instead.

@szdziedzic szdziedzic requested a review from douglowder July 7, 2026 09:11
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