Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Please upgrade mqtt.js #595

Closed
gilbertwong96 opened this issue Jun 7, 2021 · 1 comment
Closed

Please upgrade mqtt.js #595

gilbertwong96 opened this issue Jun 7, 2021 · 1 comment
Assignees
@oceanlvr
Labels
bug Something isn't working

Comments

@gilbertwong96
Copy link
Contributor

Here is the bug report of MQTT.js which is bumped v4.2.1: mqttjs/MQTT.js#1152.

And here is the bug fix PR: mqttjs/MQTT.js#1162.

I hope you guys can follow the upstream of MQTT.js and fix bug asap.
@gilbertwong96 gilbertwong96 added the bug Something isn't working label Jun 7, 2021
gilbertwong96 added a commit to gilbertwong96/MQTTX that referenced this issue Jun 7, 2021
@gilbertwong96
fix(dependency): Upgrade MQTT.js to fix reconnect bug
62947c4 
References:
* mqttjs/MQTT.js#1152
* mqttjs/MQTT.js#1162
* emqx#595
@gilbertwong96 gilbertwong96 mentioned this issue Jun 7, 2021
Closed
gilbertwong96 added a commit to gilbertwong96/MQTTX that referenced this issue Jun 7, 2021
@gilbertwong96
fix(dependency): Upgrade MQTT.js to fix reconnect bug
ba94a33 
References:
* mqttjs/MQTT.js#1152
* mqttjs/MQTT.js#1162
* emqx#595
@gilbertwong96 gilbertwong96 mentioned this issue Jun 7, 2021
Merged
@ysfscream
Copy link
Member

Hi, thanks again! It is now fixed at https://github.com/emqx/MQTTX/releases/tag/v1.6.0.

@ysfscream ysfscream moved this to Done in MQTTX May 25, 2023
@ysfscream ysfscream added this to MQTTX May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oceanlvr oceanlvr

Labels
bug Something isn't working
Projects
MQTTX
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(dependency): Upgrade MQTT.js to fix reconnect bug gilbertwong96/MQTTX
3 participants
@gilbertwong96 @ysfscream @oceanlvr