Skip to content

fix: upgrade nestjs dependencies to upgrade multer transitive import#18374

Merged
charlesBochet merged 1 commit intomainfrom
dependabot-549
Mar 4, 2026
Merged

fix: upgrade nestjs dependencies to upgrade multer transitive import#18374
charlesBochet merged 1 commit intomainfrom
dependabot-549

Conversation

@mabdullahabaid
Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​nestjs/​core@​11.1.9 ⏵ 11.1.159310010095 +2100
Updated@​nestjs/​common@​11.1.9 ⏵ 11.1.159910010095 +5100
Updated@​nestjs/​testing@​11.1.9 ⏵ 11.1.1510010010095 +5100
Updated@​nestjs/​platform-express@​11.1.9 ⏵ 11.1.15100 +2100100100 +6100

View full report

@mabdullahabaid mabdullahabaid self-assigned this Mar 4, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 4, 2026

Greptile Summary

This PR upgrades @nestjs/common, @nestjs/core, @nestjs/platform-express, and @nestjs/testing from 11.1.9 to 11.1.15 in order to transitively pull in multer 2.1.0, which resolves Dependabot security alerts 549 and 550. The upgrade is confined to the twenty-server package and brings in several other minor version bumps across the transitive dependency graph (express, cors, body-parser, file-type, token-types, etc.).

  • @nestjs/platform-express 11.1.15 now depends on multer 2.1.0 (was 2.0.2), which removes the vulnerable mkdirp, object-assign, and xtend transitive packages from multer's direct dependencies.
  • @nestjs/common 11.1.15 pulls in file-type 21.3.0, which in turn drops the fflate dependency via @tokenizer/inflate 0.4.1.
  • All four NestJS packages are updated consistently and their peer-dependency constraints (^11.0.0) are still satisfied by the rest of the NestJS ecosystem in this repo.
  • The yarn.lock checksums are all updated correctly and the dependency graph is internally consistent.

Confidence Score: 5/5

  • This PR is safe to merge — it is a targeted security dependency upgrade with no application logic changes.
  • All changes are confined to dependency version pins and the generated lock file. The version bumps are patch/minor releases within the same major version (11.x), peer-dependency constraints remain satisfied, and the multer upgrade directly addresses the stated CVEs. No application code was modified.
  • No files require special attention.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["@nestjs/platform-express\n11.1.9 → 11.1.15"] --> B["multer\n2.0.2 → 2.1.0 ✅"]
    A --> C["express\n5.1.0 → 5.2.1"]
    A --> D["cors\n2.8.5 → 2.8.6"]
    C --> E["body-parser\n2.2.1 → 2.2.2"]
    E --> F["qs\n6.14.0 → 6.14.1+"]
    G["@nestjs/common\n11.1.9 → 11.1.15"] --> H["file-type\n21.1.0 → 21.3.0"]
    H --> I["@tokenizer/inflate\n0.3.1 → 0.4.1\n(drops fflate dep)"]
    H --> J["token-types\n6.1.1 → 6.1.2"]
    J --> K["@borewit/text-codec\n0.1.1 → 0.2.1"]
    B -- "Resolves" --> L["Dependabot Alert 549\nDependabot Alert 550"]
    style L fill:#d4edda,stroke:#28a745
    style B fill:#d4edda,stroke:#28a745
Loading

Last reviewed commit: 9996b5a

@FelixMalfait
Copy link
Copy Markdown
Member

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:27747

This environment will automatically shut down after 5 hours.

@charlesBochet charlesBochet merged commit 2f9c94d into main Mar 4, 2026
78 checks passed
@charlesBochet charlesBochet deleted the dependabot-549 branch March 4, 2026 09:34
@twenty-eng-sync
Copy link
Copy Markdown

Hey @mabdullahabaid! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

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.

3 participants