stream: function() support doesn't work as documented.

The docs say that the function is async and should call a callback upon creation of the stream.

  stream: function (cb) {
    const newStream = new SocksConnection({host: 'remote.host', port: 3306}, {host: 'localhost', port: 1080});
    cb(null, newStream);
  }

The code says the function is synchronous and should return the stream.

      // if stream is a function, treat it as "stream agent / factory"
      if (typeof opts.config.stream == 'function') {
        this.stream = opts.config.stream(opts);
      } else {

[sidorares]

So you mean creating a duplex, returning that, and then connecting the new stream once its created asynchronously?

yes, something like that. I’m also happy to try to add support for Promise<Stream> result

[sidorares]

you can still do it lazily - return first “half” of the stream synchronously and connect events later when you have downstream connection, though it’s not the best experience