Multer проверка наличия параметра file?

Question

[Богдан]

Здравствуйте, а не подскажите как правильно сделать валидацию в multer, что если нет наличия параметра file выдавать ошибку?

const fileFilter = async (req, file, next) => {
 ...
  return next(null, true);
};

const storage = multer.diskStorage({
  destination: (req, file, cb) => cb(null, pathGalleryCategories),
  filename: (req, file, cb) => cb(null, req.categoryId.toString())
});

const upload = multer({
  fileFilter,
  storage
});

routes.post('', upload.single('file'), (req, res) => {
  res.json({ id: req.categoryId });
});

Получается, если не задать параметр file, тогда вообще не срабатывает middlware upload.single('file') -> fileFilter.
То как правильно делать проверку? Спасибо.

Answer 1

[Lynn «Кофеман»]

Добавить ещё один middleware или завернуть multer в middleware

routes.post('',
  upload.single('file'),
  (req, res, next) => {
    next(req.file ? null : new Error('No file!'));
  },
  (req, res) => {
    res.json({ id: req.categoryId });
  }
);

или

routes.post('',
  (req, res, next) => {
    upload.single('file')(req, res, (err) => {
      if (err) return next(err);
      if (!req.file) return next(new Error('No file!'));
      next();
    });
  },
  (req, res) => {
    res.json({ id: req.categoryId });
  }
);

Comments

[Богдан]

Спасибо, а можно ли как-то в этой миддлваре, провалидировать параметры и создать запись в базе которая соответствует имени файла, получается, что они парсятся multer, а миддлварь получается раньше multer. Или как в этом случаи правильно сделать?

const fileFilterCreateCategory = async (req, file, next) => {
  if (!allowedTypes.includes(file.mimetype)) {
    return next(new Error('LIMIT_FILE_TYPES'));
  }

  return next(null, true);
};

const storageCreateCategory = multer.diskStorage({
  destination: (req, file, cb) => cb(null, pathGalleryCategories),
  filename: (req, file, cb) => cb(null, req.categoryId.toString())
});

const uploadCreateCategory = multer({
  fileFilter: fileFilterCreateCategory,
  limits: { fileSize: process.env.APP_MAX_FILE_SIZE },
  storage: storageCreateCategory
});

routes.post('',
  async (req, res, next) => {
    const { num, visible, name } = req.body;
    if (!(num && visible && name)) next(new Error('MISSING_PARAMS'));

    const category = await models.GalleryCategory.create({ num, visible, name });

    req.categoryId = category.id;

    next(req.file ? null : new Error('NO_FILE'));
  },
  uploadCreateCategory.single('file'),
  (req, res) => res.json({ id: req.categoryId }));

[Lynn «Кофеман»]

Богдан, ну так поменяйте их местами. Вообще непонятно откуда у вас req.body до того, как выполнился multer.

[Богдан]

Ну так body как раз и нет падает ошибка.

Вот как бы 2 варианта
1) Ну если я их поменяю местами, тогда получается, что запись в базе создастся после того как файл загрузился, а у меня id таблицы как раз соответствует имени файла.

2) У меня добавиться метод patch - частичное изменении. Тоесь фактически файл может не грузиться, тогда как быть в данном случаи? Логично получить id по которому будет идти обновление записи, проверить наличие записи в базе и уже на основании этого запускать миддварь мультера для загрузки файла?

[Lynn «Кофеман»]

Ничего не понял.
1) это же как раз логично, какой смысл создавать запись если файла нет.

Ну или сохраняйте во временный файл а потом перекладывайте куда нужно

[Богдан]

Вот создал вопрос, может подскажите? вопрос