Merge pull request #50 from PDOK/jd/ingressrouteurls-validation

jelledijkstra97 · web-flow · commit 9a53a2323f8e · 2025-06-03T13:27:33.000+02:00
Check if old url in part of ingressRouteURLs
diff --git a/api/v3/atom_validation.go b/api/v3/atom_validation.go
@@ -48,6 +48,11 @@ func (atom *Atom) ValidateUpdate(c client.Client, atomOld *Atom) ([]string, erro
 			&allErrs,
 			field.NewPath("spec").Child("service").Child("baseUrl"),
 		)
+	} else if atom.Spec.Service.BaseURL.String() != atomOld.Spec.Service.BaseURL.String() {
+		err := smoothoperatorvalidation.ValidateIngressRouteURLsContainsBaseURL(atom.Spec.IngressRouteURLs, atomOld.Spec.Service.BaseURL, nil)
+		if err != nil {
+			allErrs = append(allErrs, err)
+		}
 	}
 
 	smoothoperatorvalidation.ValidateIngressRouteURLsNotRemoved(atomOld.Spec.IngressRouteURLs, atom.Spec.IngressRouteURLs, &allErrs, nil)

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,11 @@ func (atom Atom) ValidateUpdate(c client.Client, atomOld Atom) ([]string, erro`
`48`	`48`	`&allErrs,`
`49`	`49`	`field.NewPath("spec").Child("service").Child("baseUrl"),`
`50`	`50`	`)`
	`51`	`+ } else if atom.Spec.Service.BaseURL.String() != atomOld.Spec.Service.BaseURL.String() {`
	`52`	`+ err := smoothoperatorvalidation.ValidateIngressRouteURLsContainsBaseURL(atom.Spec.IngressRouteURLs, atomOld.Spec.Service.BaseURL, nil)`
	`53`	`+ if err != nil {`
	`54`	`+ allErrs = append(allErrs, err)`
	`55`	`+ }`
`51`	`56`	`}`
`52`	`57`
`53`	`58`	`smoothoperatorvalidation.ValidateIngressRouteURLsNotRemoved(atomOld.Spec.IngressRouteURLs, atom.Spec.IngressRouteURLs, &allErrs, nil)`