Skip to content

Bug-fix for redis connection

username-removed-487110 requested to merge 113-redis-fix into master

Relates #113 (closed)

Fixes:

  1. 2017-03-02_17:32:46.77638 2017/03/02 17:32:46 error: Failed to connect to redis: dial unix: missing address
  2. Is there a reason why ReadTimeout is an integer, not a time.Duration as we do it in other places?
  • Nope, now it's a time.Duration
  1. ReadTimeout for PubSub.Receive doesn't make sense.
  • Fair enough, now using KeepAlivePeriod for PubSub.
  1. We have ReadTimeout, but I don't see equivalents for ConnectTimeout and WriteTimeout. Is this intended?
  • Support for WriteTimeout added

Merge request reports