Skip to content

Add npm overrides for vulnerable transitive packages#366

Open
arpitjain099 wants to merge 2 commits into
apache:mainfrom
arpitjain099:security/npm-overrides-serialize-and-path
Open

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

Conversation

@arpitjain099
Copy link
Copy Markdown

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
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
Author

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

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