Skip to content

Release: 2.4.0 - #1469

Open
UlisesGascon wants to merge 1 commit into
mainfrom
release/2.4.0
Open

Release: 2.4.0#1469
UlisesGascon wants to merge 1 commit into
mainfrom
release/2.4.0

Conversation

@UlisesGascon

Copy link
Copy Markdown
Member

Blockers

We need/want to land (waiting for reviews):

What's included in the CHANGELOG.md

## 2.4.0

- Add `filename` to `LIMIT_FILE_SIZE` and `LIMIT_UNEXPECTED_FILE` errors ([#1416](https://github.com/expressjs/multer/pull/1416))
- Accept a function for `limits`, called with the request, to set limits per request ([#1133](https://github.com/expressjs/multer/pull/1133))
- Add opt-in `flush` option to `DiskStorage` to fsync files before the callback runs ([#1458](https://github.com/expressjs/multer/pull/1458))
- Reject non-integer or negative `limits` values at construction time; a float limit silently disabled the check ([#1395](https://github.com/expressjs/multer/pull/1395), [#1335](https://github.com/expressjs/multer/pull/1335))
- Accept requests with exactly `limits.parts` parts; `LIMIT_PART_COUNT` now fires only when the limit is exceeded. If you set `parts` one higher to work around this, you can drop the extra one ([#1446](https://github.com/expressjs/multer/pull/1446))
- Files skipped by `fileFilter` no longer count towards `maxCount` ([#1426](https://github.com/expressjs/multer/pull/1426))
- Change the `LIMIT_UNEXPECTED_FILE` message to "Unexpected file field" ([#426](https://github.com/expressjs/multer/pull/426))
- Remove the `concat-stream` dependency ([#1356](https://github.com/expressjs/multer/pull/1356))
- Docs: remove the translated READMEs ([#1463](https://github.com/expressjs/multer/pull/1463))
- Internal: run the test suite on macOS ([#1464](https://github.com/expressjs/multer/pull/1464))

What's Changed

New Contributors

Full Changelog: v2.3.0...main

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 99.739%. remained the same — release/2.4.0 into main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants