BUG/MEDIUM: ssl-passthrough: Fix default delay value

When no client-timeout is provided inspect delay rule should fallback to a default value. Reported in #414 Regression introduced in 6333c37 Backport to 1.7
haproxytech · Jan 31, 2022 · c5313bc · c5313bc
1 parent 0b1be5c
commit c5313bc
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/controller/handler/https.go b/controller/handler/https.go
@@ -260,8 +260,10 @@ func (h HTTPS) toggleSSLPassthrough(passthrough bool, cfg *config.ControllerCfg,
 
 func (h HTTPS) sslPassthroughRules(k store.K8s, cfg *config.ControllerCfg) error {
 	inspectTimeout, err := annotations.Timeout("timeout-client", k.ConfigMaps.Main.Annotations)
-	if err != nil {
-		logger.Errorf("SSL Passthrough: %s", err)
+	if inspectTimeout == nil {
+		if err != nil {
+			logger.Errorf("SSL Passthrough: %s", err)
+		}
 		inspectTimeout = utils.PtrInt64(5000)
 	}
 	errors := utils.Errors{}