Skip to content

fix: upgrade multer 1.x to 2.x, replace inMemory with memoryStorage - #1

Open
dreyesb wants to merge 2 commits into
masterfrom
fix/critical-dependencies
Open

fix: upgrade multer 1.x to 2.x, replace inMemory with memoryStorage#1
dreyesb wants to merge 2 commits into
masterfrom
fix/critical-dependencies

Conversation

@dreyesb

@dreyesb dreyesb commented Jul 23, 2026

Copy link
Copy Markdown

multer 2.0 removed the inMemory option. Replace with storage: multer.memoryStorage() and update the multer require to use the constructor returned by require('multer').

dreyesb and others added 2 commits July 23, 2026 13:30
multer 2.0 removed the inMemory option. Replace with storage: multer.memoryStorage()
and update the multer require to use the constructor returned by require('multer').

Co-Authored-By: GitHub Copilot <copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

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 updates the runner’s multipart/form-data handling to be compatible with Multer 2.x (which removed the inMemory option) and bumps Lodash to a newer major version with the required API adjustment.

Changes:

  • Upgrade multer to ^2.0.0 and switch from inMemory: true to storage: multer.memoryStorage().
  • Update Multer initialization to reuse a single require('multer') import and call it as a constructor.
  • Upgrade lodash to ^4.18.1 and replace _.any with _.some.

Reviewed changes

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

File Description
package.json Bumps multer to 2.x and lodash to 4.x.
fittings/swagger_raw.js Updates Lodash API usage (_.any_.some) for compatibility with Lodash 4.x.
fittings/swagger_params_parser.js Updates Multer configuration/initialization for Multer 2.x and uses memoryStorage().

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

Comment thread fittings/swagger_params_parser.js
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.

4 participants