Matt Brittan

While [`SetKeepAlive`](https://github.com/eclipse/paho.mqtt.golang/blob/v1.3.5/options.go#L245) accepts a `time.Duration` this is converted into seconds internally. When working out when to make the next check it used [the following](https://github.com/eclipse/paho.mqtt.golang/blob/v1.3.5/ping.go#L34): ``` if c.options.KeepAlive > 10 {...

Changes have been merged but I'll leave this issue open for any issues arising.

I've been running this live in four systems (some with very poor connections) for a couple of weeks without experiencing any issues. Will leave the issue open until after the...

I'm not sure that there is a solution to this unfortunately; as per [the spec](http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc384800415): >If a Server implementation does not authorize a PUBLISH to be performed by a Client;...

No worries - I have amended the title so that it more clearly covers what I believe you are requesting. The initial reconnection attempt needs to be immediate (because the...

Would be happy to accept a PR that implements this (I think most users, myself included, connect to one broker and the logging works OK in that situation). My preference...

Please point out where in this library the memory leak is? The comment you pasted does not say that `time.After` causes a memory leak; just that resources are not freed...

Why are you preferring ` time.NewTicker` over `time.NewTimer`? (a ticker will fire periodically, a timer will fire once). The change should be fairly simple: ``` t := time.NewTimer(time.Second * 2)...

Are you sure you are using version 1.3.5? (I note that the library is vendored in your project so the version in use may not match your `go.mod`). The line...

Apologies - I was looking at `master` and changes to the copyright notices have pushed everything around (thought I checked the right tag this morning but was mistaken). So looking...