parseFormData crashes node express server when exceeding max size

[lasseklovstad]

When uploading a file that is too large the parseFormData throws an MultipartParseError that is uncatchable. When it is thrown in a Remix action function, Remix is unable to catch it so the server crashes :(

const formData = await parseFormData(
    request,
    createTempUploadHandler(),
    { maxFileSize: 20 * 1024 * 1024 },
  );

MultipartParseError: File size exceeds maximum allowed size of 20971520 bytes
    at MultipartParser.#writeBody (file:///C:/projects/pils/node_modules/@mjackson/multipart-parser/dist/lib/multipart.js:240:25)
    at MultipartParser.#write (file:///C:/projects/pils/node_modules/@mjackson/multipart-parser/dist/lib/multipart.js:173:40)
    at MultipartParser.parse (file:///C:/projects/pils/node_modules/@mjackson/multipart-parser/dist/lib/multipart.js:123:28)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)

[Bessonov]

I'm not sure if I encountered the same error. I use parseMultipartRequest to retrieve files from Node's request object. When I set maxFileSize, I can catch the error and respond with a 422 status. However, after this, the browser's next request to another endpoint hangs, although opening a new tab works as expected. There is also a proxy in-between, so not sure it comes from the library.

If I let maxFileSize to Infinity and stop processing while reading chunks from file.body, everything works as expected. It seems like something might not be terminated, closed, or ended properly. However, just by looking at the source code of multipart.ts, I don't see anything obviously wrong 🤷

[mattoni]

I'm also encountering this issue.

[mattoni]

I am only able to prevent my server from crashing by adding

process.on("unhandledRejection", (reason, promise) => {
	logger.error("Unhandled rejection at:", promise, "reason:", reason);
});

but that isn't ideal since the error is no longer being handled within the action, so we can't gracefully return a message to the user.

[Apsysikal]

I'm having the same issue currently. Couldn't get it to catch the error, no matter where i add the catch clause.
Did anyone have any luck in fixing it?