Merge pull request #785 from cfbraun/master

fix: reduce StaleKeyLifetimeSec to 4 minutes

Author: foxcpp

internal/target/remote/remote.go

@@ -146,7 +146,7 @@ func (rt *Target) Init(cfg *config.Map) error {
 		MaxKeys:             5000,
 		MaxConnsPerKey:      5,      // basically, max. amount of idle connections in cache
 		MaxConnLifetimeSec:  150,    // 2.5 mins, half of recommended idle time from RFC 5321
-		StaleKeyLifetimeSec: 60 * 5, // should be bigger than MaxConnLifetimeSec
+		StaleKeyLifetimeSec: 60 * 4, // make sure that cleanup runs before recommended idle time from RFC 5321
 	}
 	cfg.Int("conn_max_idle_count", false, false, 5, &poolCfg.MaxConnsPerKey)
 	cfg.Int64("conn_max_idle_time", false, false, 150, &poolCfg.MaxConnLifetimeSec)

internal/target/remote/remote_test.go

@@ -64,7 +64,7 @@ func testTarget(t *testing.T, zones map[string]mockdns.Zone, extResolver *dns.Ex
 			MaxKeys:             5000,
 			MaxConnsPerKey:      5,      // basically, max. amount of idle connections in cache
 			MaxConnLifetimeSec:  150,    // 2.5 mins, half of recommended idle time from RFC 5321
-			StaleKeyLifetimeSec: 60 * 5, // should be bigger than MaxConnLifetimeSec
+			StaleKeyLifetimeSec: 60 * 4, // make sure that cleanup runs before recommended idle time from RFC 5321
 		}),
 	}