Return stream instance synchronously

[TheLudd]

In the documentation, this is declared:

var webshot = require('webshot');
var fs      = require('fs');

webshot('google.com', function(err, renderStream) {
  var file = fs.createWriteStream('google.png', {encoding: 'binary'});

  renderStream.on('data', function(data) {
    file.write(data.toString('binary'), 'binary');
  });
});

The stream instance is returned asynchronously. Why is this? Should the code not look like this?

var webshot = require('webshot');
var fs      = require('fs');
var file = fs.createWriteStream('google.png', {encoding: 'binary'});

var renderStream = webshot('google.com');
renderStream.pipe(file);

[Wizek]

👍

[lukepolo]

Its because it spins up a instance of phantom

[OleMchls]

I stumbled upon this today as well, if i understand the code correctly returning the writable stream sync should not a problem

node-webshot/lib/webshot.js

Line 245 in ce4b863

cb(null, s);

should be

cb(null, s);
return s;