Unexpected identifier error in browserifying mqtt

[mayooran99]

I need to use the mqtt client from the browser and I followed the following steps as mentioned in the doc.

And then created a html page as specified in the doc.

When I ran the commands, the browserMqtt.js got created successfully. But when I run the HTML I am getting the following error.

Is there something wrong with the library?

[sandro-k]

@mayooran99 could you pleas expand the error given in the Console. Just copy-paste the stacktrace and the version of Chrome you are using.

[mcollina]

There is something wrong with the browserify version you are using:
I got this line:

  if(length === -1 || end > this._list.length || end > this.packet.length)

[mcollina]

I checked this here, and I get the same output, not sure what's up to.

@sandro-k can you have a look?

[sandro-k]

@mayooran99 and @mcollina I can confirm that this is a bug in https://github.com/mqttjs/mqtt-packet/blob/master/parser.js#L355 and https://github.com/mqttjs/mqtt-packet/blob/master/parser.js#L370. Somehow a char 160 ended up after if(length === -1 || end > this._list.length ||

I will create a PR

[sandro-k]

@mayooran99 thanks for reporting this.

[sandro-k]

I can confirm when forcing "mqtt-packet": "^3.4.7" I can now bundle and use mqtt.js in the browser without any issue.

[mcollina]

I don't think there is a need to do a release, as we depend on mqtt-packet with the ^.
If anyone faces this issue, just zap your node_modules folder and reinstall.