How can I use Axios to get an array of files to give to Pizzip to generate a .zip archive?

Question

Here's the documentation for Pizzip for those unfamiliar: https://github.com/open-xml-templating/pizzip/tree/master/documentation

Also, I'm using VueJS (2), ES6, TS

I am trying to download individual files with Axios and then put them into a zip file using Pizzip.

async getFile(mat: Material): Promise<any> {
    try {
      const file = await axios.get(mat.url, { responseType: 'arraybuffer' })
      return file.data
    } catch (error) {
      console.log(error)
      return {}
    }
}

This is giving me the ArrayBuffer correctly. Then I have the following:

async getZip() {
    let zip = new pizzip()
    this.materials.forEach(async mat => {
        const record = await PortalService.getFile(mat)
        console.log(record) // logs the ArrayBuffer
        zip.file(mat.name, record)
    })
    let files = zip.generate({ type: 'blob' })
    return saveAs(files, `${this.request.id}-files.zip`)
}

Pizzip docs said that the data argument of zip.file() can be of type ArrayBuffer, but when I download the zip it's just an empty archive. I've tried using the optional options parameter for zip.file(), but nothing has worked. Thank you in advance.

Can you post the code you have tried so far and which isn't working? — Decade Moon, Apr 02 '20 at 22:19
@DecadeMoon sorry about that, I accidentally submitted the post before finishing it. — Monica, Apr 02 '20 at 22:28

score 1 · Accepted Answer · answered Apr 02 '20 at 23:00

The issue is probably to do with the async for-loop. The forEach method does not support async loop functions; it will just execute each iteration without waiting for the previous to finish. So basically you're generating the zip file before any files had been added to it.

There are two ways you can handle this:

Fire all requests concurrently and wait for them all to finish. This will be faster since it is sending the requests concurrently, but be careful not to spam hundreds of requests at once!

await Promise.all(this.materials.map(async mat => {
  const record = await PortalService.getFile(mat)
  zip.file(mat.name, record)
}))

Fire each request sequentially, one at a time. Slower, but doesn't spam requests.

for (const mat of this.materials) {
  const record = await PortalService.getFile(mat)
  zip.file(mat.name, record)
}

Thank you, I will try these! EDIT: Worked perfect, thank you very much! — Monica, Apr 02 '20 at 23:17

How can I use Axios to get an array of files to give to Pizzip to generate a .zip archive?

1 Answers1