Skip to content

Conversation

@addaleax
Copy link
Collaborator

@addaleax addaleax commented Dec 3, 2025

Resolves a CI vuln check failure for SNYK-JS-EXPRESS-14157151. We are not affected by this vulnerability, and the CVE was even revoked, but this is still the easiest way to resolve this warning.

Resolves a CI vuln check failure for SNYK-JS-EXPRESS-14157151.
We are not affected by this vulnerability, and the CVE was even
revoked, but this is still the easiest way to resolve this warning.
@addaleax addaleax requested a review from a team as a code owner December 3, 2025 12:01
Copilot AI review requested due to automatic review settings December 3, 2025 12:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR upgrades Express to version 5.2.0 to resolve a CI vulnerability check failure (SNYK-JS-EXPRESS-14157151), even though the project is not actually affected by the vulnerability and the CVE was revoked. The upgrade allows removal of dependency overrides that were previously needed.

Key Changes:

  • Removes overrides section from package.json containing cookie and body-parser version pins

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"cookie": "^0.7.2",
"body-parser": "^2.2.1"
}
]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Don't need these anymore, the updated version of express already allows these ranges

@addaleax addaleax added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Dec 3, 2025
@addaleax addaleax changed the title chore(deps): bump express to 5.2.0 chore(deps): bump express to 5.2.0 MONGOSH-3047 Dec 3, 2025
@addaleax addaleax changed the title chore(deps): bump express to 5.2.0 MONGOSH-3047 chore(deps): bump express to 5.2.1 MONGOSH-3047 Dec 3, 2025
@addaleax addaleax merged commit 6fc1946 into main Dec 4, 2025
149 of 156 checks passed
@addaleax addaleax deleted the express-5.2.0 branch December 4, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants