do not set connected before the connection is established with the server

[rolandjitsu]

May I ask why is the self.connected set to YES when the method connectWithCompletionHandler:(void (^)(MQTTConnectionReturnCode code))completionHandler; is called on line 212 in MQTTKit.m?

I would expect that to only be set to true or false when a connection is made with the server or closed, not before is opened as it happens in this case. Or is that intended behaviour?

Anyway, great work and very useful 👍

[jmesnil]

that was a silly mistake (I've pushed a fix before I notice your PR).

Thanks for pointing it out

[rolandjitsu]

No problem @jmesnil :) Just go ahead and close the other one as well.