Skip to content

Add npm overrides for vulnerable transitive packages#366

Merged
tuhaihe merged 2 commits into
apache:mainfrom
arpitjain099:security/npm-overrides-serialize-and-path
May 29, 2026
Merged

Add npm overrides for vulnerable transitive packages#366
tuhaihe merged 2 commits into
apache:mainfrom
arpitjain099:security/npm-overrides-serialize-and-path

Conversation

@arpitjain099
Copy link
Copy Markdown
Contributor

Summary

  • Add package manager overrides for two vulnerable transitive dependencies:
    • serialize-javascript -> 7.0.5
    • path-to-regexp -> 1.9.0
  • Refresh package-lock.json accordingly.

Why

Dependabot currently reports high-severity advisories for these transitive packages. This keeps the existing direct dependency set while forcing patched versions in the dependency tree.

Validation

  • npm install
  • npm run build
  • npm audit --json now reports a reduced high-severity count in this environment (from 18 to 12).

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hiiii, @arpitjain099 welcome!🎊 Thanks for taking the effort to make our project better! 🙌 Keep making such awesome contributions!

@arpitjain099 arpitjain099 force-pushed the security/npm-overrides-serialize-and-path branch from 28dca36 to 9fc3548 Compare May 13, 2026 17:13
@arpitjain099
Copy link
Copy Markdown
Contributor Author

Hi @tuhaihe, gentle ping on this. PR has been open for 4 days without review. I noticed you've been on the recent-merger side of recent merges in this repo. When you have a moment, would you mind giving it a quick look? No urgency. Happy to address any feedback.

@tuhaihe
Copy link
Copy Markdown
Member

tuhaihe commented May 18, 2026

Hi @tuhaihe, gentle ping on this. PR has been open for 4 days without review. I noticed you've been on the recent-merger side of recent merges in this repo. When you have a moment, would you mind giving it a quick look? No urgency. Happy to address any feedback.

NP. Let me review it later. Thanks again!

@arpitjain099
Copy link
Copy Markdown
Contributor Author

Friendly ping in case this slipped off the queue. Happy to rebase if needed.

@tuhaihe tuhaihe merged commit c5cf7ef into apache:main May 29, 2026
2 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.

2 participants