使用multer将多个文件发送到Google存储桶-为什么我的文件为空?

时间:2019-03-01 16:23:38

标签: javascript node.js google-cloud-storage multer

我以前只设置了一个文件上传并正常工作。现在,我需要使其处理多个文件。

这是我的代码:

const multer = require('multer')
const { Storage } = require('@google-cloud/storage')

const storage = new Storage()

const m = multer({ storage: multer.memoryStorage()  })

module.exports = app => {
  app.use('/', router)
  router.post(
    '/reader-:shortId/file-upload',
    passport.authenticate('jwt', { session: false }),
    m.array('files'),
    async function (req, res) {
      const bucketName = req.params.shortId.toLowerCase()
      await storage.createBucket(bucketName)
      bucket = storage.bucket(bucketName)
      let promises = []

      req.files.forEach((file) => {
        const blob = bucket.file(file.originalname)
        const newPromise =  new Promise((resolve, reject) => {
          blob.createWriteStream({
            metadata: { contentType: file.mimetype }
          }).on('finish', async response => {
            await blob.makePublic()
            resolve(response)
          }).on('error', err => {
            reject('upload error: ', err)
          }).end()
        })
       promises.push(newPromise)
     })

     Promise.all(promises).then((response) => {
       // the response I get here is [undefined, undefined]
       res.status(200).send(response)
     }).catch((err) => {
       res.status(400).send(err.message)
     });
    })
  }

req.files确实给了我一个文件数组,带有一个缓冲区和一个有意义的大小。 承诺全部解决。 但是,一旦我检查了Google存储桶中的文件,它们的名称就正确了,但是没有任何内容(大小为0)

正如我之前所说,当我使用一个文件(使用m.single('file')

我不想将存储桶用作multer设置的目标,因为在上传到Google存储桶之前,我还必须更改文件名。

编辑:这是Google云文档为单个文件上传(https://cloud.google.com/nodejs/getting-started/using-cloud-storage)给出的代码示例:

function sendUploadToGCS (req, res, next) {
  if (!req.file) {
    return next();
  }

  const gcsname = Date.now() + req.file.originalname;
  const file = bucket.file(gcsname);

  const stream = file.createWriteStream({
    metadata: {
      contentType: req.file.mimetype
    },
    resumable: false
  });

  stream.on('error', (err) => {
    req.file.cloudStorageError = err;
    next(err);
  });

  stream.on('finish', () => {
    req.file.cloudStorageObject = gcsname;
    file.makePublic().then(() => {
      req.file.cloudStoragePublicUrl = getPublicUrl(gcsname);
      next();
    });
  });

  stream.end(req.file.buffer);
}

我最初有类似的工作方式,但是我只是不知道它从哪里获取文件缓冲区数据。这可能与多个文件有所不同。

3 个答案:

答案 0 :(得分:1)

我知道为时已晚,但是有人可能会想在Google Cloud Storage上上传多个文件寻找答案。

依赖项:

  • 快递
  • Google Cloud库
  • 马尔特
  • Body Parser

这是控制器代码。

exports.post_image_upload = async (req, res) => {
   /** Check if file exist */
   if (!req.files) {
       res.status(400).send('No file uploaded.');
       return;
   }

   let PublicUrls = []

   req.files.forEach((file) => {
       const blob = bucket.file(file.fieldname + '-' + Date.now() + path.extname(file.originalname))

       const blobStream = blob.createWriteStream({
           metadata: { contentType: file.mimetype }
       })
       blobStream.on('finish', ()=> {
           blob.makePublic()           
       })
       blobStream.on('error', err => {
           //Put your error message here
       })
       blobStream.end(file.buffer)

       const Url = `https://storage.googleapis.com/${bucket.name}/${blob.name}`

       PublicUrls.push(Url)
   })

   res.send(PublicUrls)
}

祝你好运

答案 1 :(得分:0)

好吧,原来我不得不改变 .end().end(file.buffer)

答案 2 :(得分:0)

玛丽·佩莱捷(Marie Pelletier),我认为您的方法是100%正确的。我修改了一些您的代码,试图避免异步响应:

let promises = []
req.files.forEach((file) => {
  const blob = bucket.file(file.originalname)
  const newPromise =  new Promise((resolve, reject) => {
    blob.createWriteStream({
      metadata: { contentType: file.mimetype },
      resumable: false //Good for small files
    }).on('finish', () => {
      const Url = `https://storage.googleapis.com/${bucket.name}/${blob.name}`;
      resolve({ name: file.originalname, url: Url });
    }).on('error', err => {
      reject('upload error: ', err);
    }).end(file.buffer);
  })
  promises.push(newPromise);
})

Promise.all(promises).then((response) => {
  res.status(200).send(response)
}).catch((err) => {
  res.status(400).send(err.message)
});

这样,我再也不会得到“未定义”了。