Multer error fieldname

[MegaSpaceHamlet]

PR Checklist

Please check if your PR fulfills the following requirements:

• [x ] The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Other... Please describe:

What is the current behavior?

When transforming a MulterError into a Nest error, the optional field property is dropped.

Issue Number: N/A

What is the new behavior?

The field property is included in the error message, if applicable.

Does this PR introduce a breaking change?

• Yes
• No

Other information

If Multer throws an error, some identifying information is hidden from the error message.

https://github.com/expressjs/multer/blob/aa42bea6ac7d0cb8fcb279b15a7278cda805dc63/lib/multer-error.js#L19. This is where Multer adds the optional field property.

[coveralls]

Pull Request Test Coverage Report for Build aa3c3f7d-478d-43ae-b28d-1cef4587eccd

Details

• 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.002%) to 92.125%

---

Totals
Change from base Build eacd3e56-4bed-4f5e-9bab-412a10335aab:	0.002%
Covered Lines:	6738
Relevant Lines:	7314

---

💛 - Coveralls

[benjGam]

It doesn't look as a "refactor" change.

[MegaSpaceHamlet]

What would be a better way to classify it?

[benjGam]

What would be a better way to classify it?

I would personally classify it as a "Bugfix" or "Feature" determining it on the base of the first idea with which this system was developed But here, i'm not able to tell if you fixed a bug or implement a new feature, so i can't tell. But i have the feeling that do not match "Refactor" criteria.