CharlesHolbrow / synk

support syncing data with browser
0 stars 0 forks source link

RedisConnection needs to handle connection errors #8

Open CharlesHolbrow opened 7 years ago

CharlesHolbrow commented 7 years ago

Right now NewConnection() in RediConnection.go gets a connection from the Pool, and uses exclusively that connection to send messages to Redis/Clients.

However, the gouroutine that pumps messages to the connection does not handle errors.

CharlesHolbrow commented 7 years ago

This is also made slightly more complicated https://github.com/CharlesHolbrow/pagen/issues/7