fastify / fastify-multipart

Multipart support for Fastify
MIT License
479 stars 103 forks source link

Strange behaviour with request.file() when filename in form is not specified #504

Open giovanni-bertoncelli opened 8 months ago

giovanni-bertoncelli commented 8 months ago

Prerequisites

Fastify version

4.23.2

Plugin version

8.1.0

Node.js version

20.x

Operating system

macOS

Operating system version (i.e. 20.04, 11.3, 10)

Sonoma 14.2

Description

I encountered a strange behavior in the request.file() method when I send a multipart (using form-auto-content) without specifying the filename of the sent file. The method returns undefined. I do not know if it is a form-auto-content issue, if this is the expected behaviour or not. If it is expected I'll close this.

Steps to Reproduce

Reproduction example: https://codesandbox.io/p/devbox/fastify-multipart-filename-nxnqgd

Expected Behavior

I expect an error (if multipart with part missing the filename are considered malformed) or otherwise the parsed file.

gurgunday commented 8 months ago

For now, we definitely don’t throw in many cases where we maybe should, this could be an interesting change that I support

Silent errors should be opt-in