Chill the defaults, we don't need that many connections

Author: VojtechVitek

config.go

@@ -20,7 +20,7 @@ type Config struct {
 	// Timeout for each Redis command after which we fall back to a local
 	// in-memory counter. If Redis does not respond within this duration,
 	// the system will use the local counter unless it is explicitly disabled.
-	FallbackTimeout time.Duration `toml:"fallback_timeout"` // default: 50ms
+	FallbackTimeout time.Duration `toml:"fallback_timeout"` // default: 100ms
 
 	// Client if supplied will be used and the below fields will be ignored.
 	//
@@ -31,6 +31,6 @@ type Config struct {
 	Port      uint16        `toml:"port"`
 	Password  string        `toml:"password"`   // optional
 	DBIndex   int           `toml:"db_index"`   // default: 0
-	MaxIdle   int           `toml:"max_idle"`   // default: 4
-	MaxActive int           `toml:"max_active"` // default: 8
+	MaxIdle   int           `toml:"max_idle"`   // default: 5
+	MaxActive int           `toml:"max_active"` // default: 10
 }

httprateredis.go

@@ -38,8 +38,13 @@ func NewRedisLimitCounter(cfg *Config) (*redisCounter, error) {
 		cfg.PrefixKey = "httprate"
 	}
 	if cfg.FallbackTimeout == 0 {
-		// Activate local in-memory fallback fairly quickly, as this would slow down all requests.
-		cfg.FallbackTimeout = 100 * time.Millisecond
+		if cfg.FallbackDisabled {
+			cfg.FallbackTimeout = time.Second
+		} else {
+			// Activate local in-memory fallback fairly quickly,
+			// so we don't slow down incoming requests too much.
+			cfg.FallbackTimeout = 100 * time.Millisecond
+		}
 	}
 
 	rc := &redisCounter{
@@ -52,10 +57,10 @@ func NewRedisLimitCounter(cfg *Config) (*redisCounter, error) {
 	if cfg.Client == nil {
 		maxIdle, maxActive := cfg.MaxIdle, cfg.MaxActive
 		if maxIdle < 1 {
-			maxIdle = 20
+			maxIdle = 5
 		}
 		if maxActive < 1 {
-			maxActive = 50
+			maxActive = 10
 		}
 
 		rc.client = redis.NewClient(&redis.Options{