Ensure webhook validation ingress has a PathTypePrefix

kubernetes · Apr 27, 2020 · 45ce28b · 45ce28b
1 parent bae9043
commit 45ce28b
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 16 deletions.
diff --git a/internal/ingress/controller/controller.go b/internal/ingress/controller/controller.go
@@ -219,6 +219,8 @@ func (n *NGINXController) CheckIngress(ing *networking.Ingress) error {
 			toCheck.ObjectMeta.Name == ing.ObjectMeta.Name
 	}
 
+	k8s.SetDefaultPathTypeIfEmpty(ing)
+
 	ings := n.store.ListIngresses(filter)
 	ings = append(ings, &ingress.Ingress{
 		Ingress:           *ing,

diff --git a/internal/ingress/controller/store/store.go b/internal/ingress/controller/store/store.go
@@ -963,28 +963,14 @@ func toIngress(obj interface{}) (*networkingv1beta1.Ingress, bool) {
 			return nil, false
 		}
 
-		setDefaultPathTypeIfEmpty(ing)
+		k8s.SetDefaultPathTypeIfEmpty(ing)
 		return ing, true
 	}
 
 	if ing, ok := obj.(*networkingv1beta1.Ingress); ok {
-		setDefaultPathTypeIfEmpty(ing)
+		k8s.SetDefaultPathTypeIfEmpty(ing)
 		return ing, true
 	}
 
 	return nil, false
 }
-
-func setDefaultPathTypeIfEmpty(ing *networkingv1beta1.Ingress) {
-	for _, rule := range ing.Spec.Rules {
-		if rule.IngressRuleValue.HTTP == nil {
-			continue
-		}
-
-		for _, path := range rule.IngressRuleValue.HTTP.Paths {
-			if path.PathType == nil {
-				path.PathType = &defaultPathType
-			}
-		}
-	}
-}
diff --git a/internal/k8s/main.go b/internal/k8s/main.go
@@ -149,3 +149,21 @@ func NetworkingIngressAvailable(client clientset.Interface) (bool, bool) {
 
 	return runningVersion.AtLeast(version114), runningVersion.AtLeast(version118)
 }
+
+// Default path type is Prefix to not break existing definitions
+var defaultPathType = networkingv1beta1.PathTypePrefix
+
+// SetDefaultPathTypeIfEmpty sets a default PathType when is not defined.
+func SetDefaultPathTypeIfEmpty(ing *networkingv1beta1.Ingress) {
+	for _, rule := range ing.Spec.Rules {
+		if rule.IngressRuleValue.HTTP == nil {
+			continue
+		}
+
+		for _, path := range rule.IngressRuleValue.HTTP.Paths {
+			if path.PathType == nil {
+				path.PathType = &defaultPathType
+			}
+		}
+	}
+}