Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

[Snyk] Security upgrade express from 4.18.2 to 4.22.0#192

Open
acid-chicken wants to merge 1 commit intoimastodonfrom
snyk-fix-e0710d0e30725811a2a5fa765ceb34d2
Open

[Snyk] Security upgrade express from 4.18.2 to 4.22.0#192
acid-chicken wants to merge 1 commit intoimastodonfrom
snyk-fix-e0710d0e30725811a2a5fa765ceb34d2

Conversation

@acid-chicken
Copy link
Copy Markdown
Member

snyk-top-banner

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.

Snyk changed the following file(s):

  • package.json
  • yarn.lock

Note for zero-installs users

If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

Vulnerabilities that will be fixed with an upgrade:

Issue Score
high severity Allocation of Resources Without Limits or Throttling
SNYK-JS-QS-14724253
  828  

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Allocation of Resources Without Limits or Throttling

Copilot AI review requested due to automatic review settings January 2, 2026 08:42
Copy link
Copy Markdown

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 is a Snyk-generated security upgrade that updates Express from version 4.18.2 to 4.22.0 (which resolves to 4.22.1 in yarn.lock) to address a high-severity security vulnerability (SNYK-JS-QS-14724253) related to "Allocation of Resources Without Limits or Throttling" in the qs dependency with a score of 828/1000.

Key Changes

  • Express upgraded from ^4.18.2 to ^4.22.0 (resolves to 4.22.1)
  • Critical dependency update: qs package upgraded from 6.11.0 to 6.14.1
  • Multiple transitive dependencies updated to support the Express upgrade

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
package.json Updates Express version constraint from ^4.18.2 to ^4.22.0
yarn.lock Adds Express 4.22.1 entry with updated dependencies including body-parser (~1.20.3), cookie (~0.7.1), qs (~6.14.0), and numerous supporting packages; retains the old 4.18.2 entry for compatibility with other dependencies still using express@^4.17.1

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

"escape-html": "^1.0.3",
"exif-js": "^2.3.0",
"express": "^4.18.2",
"express": "^4.22.0",
Copy link

Copilot AI Jan 2, 2026

Choose a reason for hiding this comment

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

The PR title and description indicate an upgrade to Express 4.22.0, but the actual resolved version in yarn.lock is 4.22.1. While using the caret range (^4.22.0) that resolves to 4.22.1 is functionally correct, there's a mismatch between the PR description and the actual upgrade target. Consider updating the PR description to reflect that the actual version being installed is 4.22.1, not 4.22.0, to avoid confusion.

Copilot uses AI. Check for mistakes.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants