Nestjs: изображение загружено, даже если проверка тела не удалась - PullRequest
Новая
       8

Nestjs: изображение загружено, даже если проверка тела не удалась

2 голосов
/ 20 октября 2019

Прежде всего, я прошу прощения за мой слабый английский.

У меня есть метод, который принимает запрос PUT, и он получает файл и BlogModel. Когда я отправляю форму из внешнего интерфейса и проверка BlogModel не удалась, файл все еще загружается.

main.ts 

import { NestFactory } from '@nestjs/core';
import { AppModule } from './core/app.module';
import { ValidationPipe } from '@nestjs/common';
import { join } from 'path';
import { NestExpressApplication } from '@nestjs/platform-express';

async function bootstrap() {
  const app = await NestFactory.create<NestExpressApplication>(AppModule);
  app.useStaticAssets(join(__dirname, '..', 'src/public'));
  app.setBaseViewsDir(join(__dirname, '..', 'src/views'));

  app.setViewEngine('hbs');
  app.useGlobalPipes(new ValidationPipe());
  await app.listen(3000);
}
bootstrap();

метод addBlog 


  @Put()
  @UseInterceptors(FileInterceptor('thumbnail', { storage: BlogStorage }))
  addBlog(@UploadedFile() file, @Body() addBlogModel: AddBlogModel) {
    console.log(file);
  }

add-blog.model.ts 

import { IsArray, IsBoolean, IsNotEmpty, IsOptional, IsString, Length } from 'class-validator';
import { Expose } from 'class-transformer';

export class AddBlogModel {
  @IsNotEmpty()
  @IsString()
  title: string;

  @IsString()
  @Length(10, 225)
  @IsOptional()
  introduction: string;

  @IsNotEmpty()
  @IsString()
  content: string;

  @IsBoolean()
  @Expose({name: 'is_published'})
  isPublished: boolean;

  @IsArray()
  @IsNotEmpty()
  tags: string[];

  @IsString()
  @IsNotEmpty()
  category: string;
}

index.hbs 

<!DOCTYPE html>
<html>
<head>

</head>

<body>
<form id="form">
    <input name="title" id="title"/>
    <input name="content" id="content"/>
    <input type="file" name="thumbnail" id="thumbnail"/>

    <button type="submit">Submit</button>
</form>

<script src="https://code.jquery.com/jquery.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/axios/0.19.0/axios.min.js"></script>
<script type="text/javascript">
    $(document).ready(function () {
        $("#form").on('submit', function (e) {
            e.preventDefault();
            const data = $(this).serializeArray()
            const data_from_array = {}
            var formData = new FormData()

            $.map(data, function(n, i){
                formData.append(n['name'], n['value'])
            });

            const file = $('input[type="file"]')[0].files[0]

            formData.append('thumbnail', file)

            const config = {
                headers: {
                    'content-type': 'multipart/form-data'
                }
            }
            axios.put('http://localhost:3000/blogs', formData, config).then(res => {
                console.log(res)
            }).catch(err => {
                console.log(err.response)
            })
        });
    })
</script>
</body>
</html>

Я ожидаю, что файл не загружен, если проверка не удалась.

1 Ответ

0 голосов
/ 21 октября 2019

То, что здесь происходит, связано с порядком выполнения цикла запросов NestJS, а именно с тем, как перехватчики вызываются и запускаются перед тем, как создаются каналы. В этом случае вы вызываете перехватчик загрузки файла, позволяете этому коду запускаться по мере необходимости, а затем проверяете свою полезную нагрузку, поэтому даже если у вас есть недопустимая полезная нагрузка, вы все равно загружаете файл. Вы можете просмотреть перехватчик загрузки файла здесь и посмотреть, как выглядит этот код. Если вам абсолютно необходимо, чтобы загрузка файла и полезная нагрузка были в одном запросе, вы всегда можете создать свой собственный перехватчик проверки вместо конвейера и запустить эту проверку перед перехватчиком загрузки файла. В противном случае вы можете сделать им два отдельных запроса.

...