Как решить ошибку Error: Route.post() requires a callback function but got a [object Object]?

Question

[Валерий Орлов]

Доброго времени суток.
При вызове маршрута: router.post('/', upload, controller.app)
возникает следующая ошибка:

Error: Route.post() requires a callback function but got a [object Object]

router/app.js

const express = require('express')
const upload = require('../middleware/upload')
const controller = require('../controllers/app')
const router =  express.Router()

router.post('/', upload, controller.app)

module.exports = router

middleware/upload.js

const multer = require('multer')
const aws = require('aws-sdk')
const multerS3 = require('multer-s3')
const Events = require('../models/Events')

const errorHandler = require('../utils/errorHandler')
const keys = require('../config/keys')
const s3FileURL = keys.AWS_URL;

aws.config.update({
  secretAccessKey: keys.ACCESS_KEY,
  accessKeyId: keys.ACCESS_KEY_ID,
  region: keys.REGION
})

const s3 = new aws.S3();

const upload = multer({
  storage: multerS3({
    s3: s3,
    bucket: 'verstesting',
    acl: 'public-read',
    key: async function (req, file, cb) {
      const today = new Date().toISOString().slice(0, 10);
      const path = "files.devices/" + req.body.id + "/" + today + "/" + file.originalname + ".jpg";
      cb(null, path);

      try {
        const updateFile = await Events.findOneAndUpdate(
            {rid: req.body.rid},
            {$set: { filename: s3FileURL+path }}
        )
        res.status(200).json({
          message: 'ok'
        })
      } catch (e) {
        errorHandler(res, e)
      }

    }
  })
}).any();

module.exports.upload = multer({upload})

controllers/app'

const mongoose = require('mongoose')
const User= require('../models/User')
const errorHandler = require('../utils/errorHandler')

module.exports.app = async function(req, res){
switch(req.body.user) {
        case 'email':
          const userEmail = await User.findOne({email: req.body.email}).select('_id')
             if (userEmail) {
              res.status(200).json({
                message: 'User find'
              })
             }
}
}

Как я понимаю, криво определяется middleware: upload.
Как избавиться от этой ошибки?
Буду благодарен любой критике и советам по коду!

Answer 1

[Иван Шумов]

Вопросов, не читая код несколько. Начиная с того почему для статики не использовать CDN и заканчивая тем какого фига вы используете ключи в коде

Comments

[Иван Шумов]

а вот вчитываясь в код - зачем вы пишете свой код для загрузки если можете использовать api s3? если хотите сделать свое api для загрузки то сделайте api gateway и прикрутите к нему lambda

[Валерий Орлов]

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

[Иван Шумов]

Валерий Орлов, курс, видимо, плохой. В рамках AWS так не делают - за такие вещи отрывают руки в паре известных мне компаний. Вы по сути пишете API для загрузки файлов в облако. Для этого есть куча вариантов:
1. Разовый доступ на загрузку файлов напрямую в AWS S3 с жесткими политиками (bucket policy) на параметры загрузки
2. AWS API Gateway и AWS lambda в фоне. Добавьте сюда IAM Role для доступа, а API Gateway можно закрыть любой безопасностью, например OpenID JWT.

В вашем случае:
- блокируемый event-loop в nodejs
- отсутствие мониторинга
- отсутствие безопасности (и дело на в dev версии)

Так же у меня подозрения что вы медленно загружаете файлы из-за (это мои догадки) отсутствия Transfer Acceleration

Мне продолжать?)

[Валерий Орлов]

Согласен, аргументы весомые. Но говоря откровенно, методы:
1. Разового доступа
2. AWS API Gateway и AWS lambda
мне не попадались.
Не могли бы Вы посоветовать грамотный материал по ним в сети?
Хотелось бы научиться делать как надо, без корявого кода.

[Иван Шумов]

Валерий Орлов, acloud.guru