1

我正在重构一些http在 Node 中使用模块的代码来got代替使用。我尝试了以下方法：

function get(url, filePath) {
  return new Promise((resolve, reject) => {
    got.stream(url).on
        ("response", response => {
            const newFile = fs.createWriteStream(filePath);
            response.pipe(newFile);
            newFile.on("finish", () => {
              newFile.close(resolve());
            });
            newFile.on("error", err => {
              reject(err);
            });    
        }).on
        ("error", err => {
             reject(err);
        });
  });
}

该finish事件从未触发。文件 ( filePath) 使用 0 个字节创建。

newFile当我使用 Nodehttp模块时，使用的代码块是有效的。

got.stream管道到文件的正确方法是什么？

4

1 回答 1

2

根据got()文档，您希望将流直接通过管道传输到您的文件，如果您使用pipeline()它，它将收集错误并报告完成。

const pipeline = promisify(stream.pipeline);
const fsp = require('fs').promises;

function get(url, filePath) { 
    return pipeline(
        got.stream(url),
        fs.createWriteStream(filePath)
    );
}

// usage
get(...).then(() => {
    console.log("all done");
}).catch(err => {
    console.log(err);
});

仅供参考，重点got.stream()是返回一个您可以直接用作流的流，并且由于您希望它转到一个文件，您可以将该流通过管道传输到该文件。我使用pipeline()而不是.pipe()因为管道有更完整的错误处理.pipe()，虽然在非错误条件下，.pipe()也可以工作。

如果出现错误，这是一个清理输出文件的版本：

function get(url, filePath) { 
    return pipeline(
        got.stream(url),
        fs.createWriteStream(filePath)
    ).catch(err => {
         fsp.unlink(filePath).catch(err => {
             if (err.code !== 'ENOENT') {
             // trying to delete output file upon error
                 console.log('error trying to delete output file', err);
             }
         });
         throw err;
    });
}
于 2020-07-27T06:15:07.530 回答