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

fix: crash on undefined packet in ack event #17

Merged

robertsLando merged 1 commit into moscajs:master from sieukrem:master

Jul 6, 2020

Contributor

sieukrem commented Jul 6, 2020

I am observing frequent crashes of aedes-cli in my environment

openzwave
zwave2mqtt
aedes
android app MyMQTT subscribing for all topics by specifying '#'

As soon MyMQTT or openzwave are not part of the setup, runs aedes without issues.

Aedes crashes because packet is undefined.


          fix: crash on undefined ack packet

0a4c123

robertsLando approved these changes

View reviewed changes

Member

robertsLando left a comment

LGTM

robertsLando merged commit c44c64c into moscajs:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet