File Interceptor only supports one options at a time (overrides other options)

[mentos1386]

I'm submitting a...

[ ] Regression 
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

If we have two upload routes in controller, second FileInterceptor destination will override first, so only second destination will be used.

Expected behavior

Each route should store files to own destination.

Minimal reproduction of the problem with instructions

  @Post('/upload/image')
  @UseInterceptors(FileInterceptor('file', { dest: 'images/' }))
  public async uploadImage(
    @Req() req: IRequest,
    @UploadedFile() file: Express.Multer.File,
  ) {
    ...
  }

  @Post('/upload/video')
  @UseInterceptors(FileInterceptor('file', { dest: 'videos/' }))
  public async uploadVideo(
    @Req() req: IRequest,
    @UploadedFile() file: Express.Multer.File,
  ) {
  ...
  }

What is the motivation / use case for changing the behavior?

We should be able to provide different destinations (options) on different routes.

Environment

Nest version: 4.6.0

[kamilmysliwiec]

Fixed in v4.6.4 🙂

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.