Remove useless code from redis pub/sub loop
Remove useless code from redis pub/sub loop

Calling Connect() multiple times is useless, as it doesn't actually
reconnect. It is handled under the hood by rediscluster.

We also call StartPubSubHandler twice per loop iteration. It's not
particularly harmful, but it's worrying that we run it here without
checking for an error.

lonelycode/tyk,nebolsin/tyk,mvdan/tyk,mvdan/tyk,mvdan/tyk,nebolsin/tyk,lonelycode/tyk,mvdan/tyk,nebolsin/tyk,nebolsin/tyk,mvdan/tyk,mvdan/tyk,nebolsin/tyk,nebolsin/tyk,mvdan/tyk,lonelycode/tyk,nebolsin/tyk,nebolsin/tyk,mvdan/tyk