[Recommend] Let FileStream inherit fs.ReadStream #173

ahuigo · 2018-07-30T08:54:22Z

I found that busboy's FileStream inherits ReadableStream, so FileStream is missing bytesRead.

Could FileStream inherits fs.ReadStream to support bytesRead property like this:

busboy.on('file', function(fieldname, file, filename, encoding, mimetype) {
          const fs = require('fs')
          const ws = fs.createWriteStream('busboy.txt')
          const ps = file.pipe(ws)
          ps.on('finish', ()=>{
            console.log(file.bytesRead); // support bytesRead here
            console.log(ws.bytesWritten) 
            console.log(ps.bytesWritten) 
          })
    });

Full demo:

var http = require('http'),
    inspect = require('util').inspect;

var Busboy = require('busboy');

http.createServer(function(req, res) {
  if (req.method === 'POST') {
    var busboy = new Busboy({ headers: req.headers });
    busboy.on('file', function(fieldname, file, filename, encoding, mimetype) {
          const fs = require('fs')
          const ws = fs.createWriteStream('busboy.txt')
          const ps = file.pipe(ws)
          ps.on('finish', ()=>{
            console.log(file.bytesRead) 
            console.log(ws.bytesWritten) 
            console.log(ps.bytesWritten) 
          })
    });
    busboy.on('field', console.log);
    busboy.on('finish', function() {
      console.log('Done parsing form!');
      res.writeHead(303, { Connection: 'close', Location: '/' });
      res.end();
    });
    req.pipe(busboy);
  } else if (req.method === 'GET') {
    res.writeHead(200, { Connection: 'close' });
    res.end('<html><head></head><body>\
               <form method="POST" enctype="multipart/form-data">\
                <input type="file" name="filefield"><br />\
                <input type="submit">\
              </form>\
            </body></html>');
  }
}).listen(8000, function() {
  console.log('Listening for requests: http://localhost:8000');
});

The text was updated successfully, but these errors were encountered:

ahuigo mentioned this issue Jul 30, 2018

[建议]让ctx.getFileStream 支持bytesRead eggjs/egg#2860

Closed

ahuigo changed the title ~~[Recommend] Let FileStream inherits fs.ReadStream~~ [Recommend] Let FileStream inherit fs.ReadStream Jul 30, 2018

ahuigo mentioned this issue Jul 30, 2018

Let FileStream support bytesRead. #174

Open

mscdex added the feature request label Dec 19, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Recommend] Let FileStream inherit fs.ReadStream #173

[Recommend] Let FileStream inherit fs.ReadStream #173

ahuigo commented Jul 30, 2018 •

edited

[Recommend] Let FileStream inherit fs.ReadStream #173

[Recommend] Let FileStream inherit fs.ReadStream #173

Comments

ahuigo commented Jul 30, 2018 • edited

ahuigo commented Jul 30, 2018 •

edited