v2.6.0 error on connect function

[CVarisco]

Hi!
I updated version few minutes ago and now I have this error on my console (without change anything on my code base).

The error is:

This is how I create client.

client = mqtt.connect('wss://mqtt.crystal.io:12346', {
    username: ...,
    password: ...,
    clientId: ...,
    reconnectPeriod: ...,
    rejectUnauthorized: ...
});

Thanks to your work.
From 🇮🇹

[mcollina]

which browser are you using? Can you upload an example of this regression?

[CVarisco]

Browser: Google Chrome 57.0.2987.133

I don't change anything on my code.
What can I do to help you with ?

[CVarisco]

If you want, I can send you this.
https://gist.github.com/CVarisco/60f10ef15cd1523fbf627f467d962b0d

[mcollina]

@CVarisco tests are green, I can't replicate. In order to fix, I need to replicate the bug. Can you provide me with a bare-minimum example where this does not work? I need a full example that I can run to diagnose the problem.

[CVarisco]

I'm texting you on Gitter to explain the issue.
I can't send the full application, because is too large.

If you can, answer me on Gitter.
Thanks a lot

[mcollina]

This was fixed by blocking the version of readable-stream to v2.2.6, so maybe there is an issue with webpack and the changes we recently did to readable-stream.

[RangerMauve]

@CVarisco could I get a copy of your webpack config to see what might be causing issues?

[CVarisco]

@RangerMauve of course!
https://gist.github.com/CVarisco/87cc9f91ea3cbd92ad276f3fb83eb4ef
It's a little confused, sorry

Why is this closed? I see no resolution... just a roll back.

[mcollina]

@flodonsky can you upload a full example to reproduce the issue?

[yohei1126]

This can be reproduced with the Webpack example on the Readme.
https://github.com/mqttjs/MQTT.js#browser

[mcollina]

@yohei1126 there might be some other condition, that's working for me.

Anyway, this should be fixed in readable-stream@2.2.9. Wipe your node_modules folder and you should be ok.