Improve server example in readme

[eladnava]

Also, handle dropped connections (to avoid memory leak)

[mcollina]

@eladnava can you please format the code following the general conventions of the rest of the repo?

Thanks for you contribution!

[eladnava]

I will, however, I noticed that in mosca, this is taken care of automatically. I no longer have a memory leak due to clients that have passively disconnected. How were you able to overcome that in mosca?

[mcollina]

Here: https://github.com/mcollina/mosca/blob/840d5abd464c7734f126d4a66cabed29577a3357/lib/server.js#L260-L262

[eladnava]

I think that event is called only on active disconnection (client sending a disconnect packet) and not for connections that have been terminated by 3rd parties (e.g. routers)

Is there any other code that specifically handles the passive disconnection case?

[mcollina]

that event is called in any case.

[eladnava]

I ended up simply migrating to mosca, as it's actually meant to be a MQTT broker (I noticed that you don't recommend using mqtt-connection for serverside usage, only as a POC).

[eladnava]

@mcollina I'd like to take another shot at this (improving the server example in README), adding the relevant event handlers. Can I submit another PR?

[mcollina]

Please do! :)