I have the following code:

let request = require('request');
let fs = require('fs');

request.get('http://localhost:8080/report.rtf')
  .pipe(fs.createWriteStream(__dirname + '/savedDoc.rtf'));

It works well, but only if the document is successfully downloaded from the given URL. However, if there is any HTTP: 403, 404 or any other error, an empty file is saved with zero length!

How can I .pipe() this only in case of HTTP: 200 response without using any additional HEAD requests? It should be possible to do in one go!

I have the following code:

let request = require('request');
let fs = require('fs');

request.get('http://localhost:8080/report.rtf')
  .pipe(fs.createWriteStream(__dirname + '/savedDoc.rtf'));

It works well, but only if the document is successfully downloaded from the given URL. However, if there is any HTTP: 403, 404 or any other error, an empty file is saved with zero length!

How can I .pipe() this only in case of HTTP: 200 response without using any additional HEAD requests? It should be possible to do in one go!

• javascript
• node.js
• npm-request
• node-streams

Share Improve this question Follow edited Mar 14, 2018 at 12:40 ralphtheninja 134k2020 gold badges113113 silver badges122122 bronze badges asked Mar 14, 2018 at 12:27 MikserMikser 9991010 silver badges1616 bronze badges

Add a ment  | 

1 Answer 1

Sorted by: Reset to default 7

Check .statusCode before piping:

const req = request
  .get('http://localhost:8080/report.rtf')
  .on('response', function (res) {
    if (res.statusCode === 200) {
      req.pipe(fs.createWriteStream(__dirname + '/savedDoc.rtf'))
    }
  })