Apropos Fehlerbehandlung: Es ist sogar noch besser, Anforderungsfehler zu hören. Ich würde sogar validieren, indem ich den Antwortcode überprüfe. Hier wird es nur für 200 Antwortcodes als Erfolg angesehen, aber andere Codes könnten gut sein.
const fs = require('fs');
const http = require('http');
const download = (url, dest, cb) => {
const file = fs.createWriteStream(dest);
const request = http.get(url, (response) => {
// check if response is success
if (response.statusCode !== 200) {
return cb('Response status was ' + response.statusCode);
}
response.pipe(file);
});
// close() is async, call cb after close completes
file.on('finish', () => file.close(cb));
// check for request error too
request.on('error', (err) => {
fs.unlink(dest);
return cb(err.message);
});
file.on('error', (err) => { // Handle errors
fs.unlink(dest); // Delete the file async. (But we don't check the result)
return cb(err.message);
});
};
Trotz der relativen Einfachheit dieses Codes würde ich empfehlen, das Anforderungsmodul zu verwenden, da es viel mehr Protokolle (Hallo HTTPS!) Verarbeitet, die von Haus aus nicht unterstützt werden http
.
Das würde so gemacht werden:
const fs = require('fs');
const request = require('request');
const download = (url, dest, cb) => {
const file = fs.createWriteStream(dest);
const sendReq = request.get(url);
// verify response code
sendReq.on('response', (response) => {
if (response.statusCode !== 200) {
return cb('Response status was ' + response.statusCode);
}
sendReq.pipe(file);
});
// close() is async, call cb after close completes
file.on('finish', () => file.close(cb));
// check for request errors
sendReq.on('error', (err) => {
fs.unlink(dest);
return cb(err.message);
});
file.on('error', (err) => { // Handle errors
fs.unlink(dest); // Delete the file async. (But we don't check the result)
return cb(err.message);
});
};