search

moscajs / mosca

MQTT broker as a module
mosca.io
3.2k stars 508 forks source link

Also persist QoS 0 subscriptions when clean flag false #694

Closed mitsos1os closed 6 years ago

mitsos1os commented 6 years ago

@mcollina Please check what is going on with the Travis Build and it fails.. .Locally with Node v8.7.0 everything succeeds.. Thanks

mitsos1os commented 6 years ago

@mcollina Sorry to insist on the issue, but because we are using Mosca in a production environment, and this bug affects us, I would prefer to use the corrected version instead of monkey patching the module code after running, could you please check if anything else is needed for the Travis Build to work? Because I think the errors are irrelevant, since in my local environment all tests succeed.

mitsos1os commented 6 years ago

Ok just removed the comments! Thanks!

mcollina commented 6 years ago

Thanks! Releasing asap.