Download File using axios : Node.js program

download-file-axios-nodejs.js

'use strict'

const Fs = require('fs')  
const Path = require('path')  
const Axios = require('axios')

async function downloadImage () {

  const url = 'https://unsplash.com/photos/AaEQmoufHLk/download?force=true'
  const path = Path.resolve(__dirname, 'images', 'code1.jpg')

  // axios image download with response type "stream"
  const response = await Axios({
    method: 'GET',
    url: url,
    responseType: 'stream'
  })

  // pipe the result stream into a file on disc
  response.data.pipe(Fs.createWriteStream(path))

  // return a promise and resolve when download finishes
  return new Promise((resolve, reject) => {
    response.data.on('end', () => {
      resolve()
    })

    response.data.on('error', () => {
      reject()
    })
  })

}

async function Main(){
    const data = await downloadImage();
    console.log("DATA ", data);
}

Main();

@markg85 This is incorrect:

Also note that this means you must not return within the try block else the finally won't be called.

The finally block always gets called. See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/try...catch#finallystatements for reference.

Maybe you were not "awaiting" inside the try and when the finally got executed the state of your code was unexpected?

Also, no need to create a Promise at the end of the try. And, since the catch is not doing anything else than jus rethrowing the error, that can be removed too.

The simplified version:

const axios = require("axios");
const fs = require("fs");

const downloadFile = async (url, destination) => {
  const writer = fs.createWriteStream(destination, { flags: "w" });

  console.log(`Starting file download with url ${url}`);
  const response = await axios({
    method: "GET",
    url: url,
    responseType: "stream",
  });

  const res = await response.data.pipe(writer);
  console.log(`Completed download with url ${url}`);

  return res;
};

@leite08 Thank you for correcting me! This was a while ago so i don't know the reason for saying what i did there. I must have had a reason... Don't know what though.

A couple fun notes though :)

1. You are assuming default autoClose = true behavior on the write object. This default has changed over time (click history for that function) so i'd prefer to go with an explicit route in this case.
2. You always open a file, even if there would be an error on axios. I'm not sure what happens to that file in this case, i'm guessing just an empty file. I'd put the writer object below the response. That way you'd at least know the file is opened on a successful http response. It can still fail but that's much less of an edge case then you have now.
3. You're cheating 😄 The createWriteStream you use is the callback API, i explicitly tried to use the promises API (this function)

Care to go for a second try with the promises api instead? I'm just really curious what you come up with!