Immutable baseURL

Jelle Dijkstra · Jelle Dijkstra · commit 6f3bc2f05e17 · 2025-03-21T15:58:06.000+01:00
diff --git a/api/v3/wfs_validation.go b/api/v3/wfs_validation.go
@@ -34,6 +34,11 @@ func (wfs *WFS) ValidateUpdate(wfsOld *WFS) ([]string, error) {
 		reasons = append(reasons, fmt.Sprintf("%v", err))
 	}
 
+	// Check service.baseURL did not change
+	if wfs.Spec.Service.BaseURL != wfsOld.Spec.Service.BaseURL {
+		reasons = append(reasons, fmt.Sprintf("service.baseURL is immutable"))
+	}
+
 	if (wfs.Spec.Service.Inspire == nil && wfsOld.Spec.Service.Inspire != nil) || (wfs.Spec.Service.Inspire != nil && wfsOld.Spec.Service.Inspire == nil) {
 		reasons = append(reasons, fmt.Sprintf("services cannot change from inspire to not inspire or the other way around"))
 	}

Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,11 @@ func (wfs WFS) ValidateUpdate(wfsOld WFS) ([]string, error) {`
`34`	`34`	`reasons = append(reasons, fmt.Sprintf("%v", err))`
`35`	`35`	`}`
`36`	`36`
	`37`	`+ // Check service.baseURL did not change`
	`38`	`+ if wfs.Spec.Service.BaseURL != wfsOld.Spec.Service.BaseURL {`
	`39`	`+ reasons = append(reasons, fmt.Sprintf("service.baseURL is immutable"))`
	`40`	`+ }`
	`41`	`+`
`37`	`42`	`if (wfs.Spec.Service.Inspire == nil && wfsOld.Spec.Service.Inspire != nil) \|\| (wfs.Spec.Service.Inspire != nil && wfsOld.Spec.Service.Inspire == nil) {`
`38`	`43`	`reasons = append(reasons, fmt.Sprintf("services cannot change from inspire to not inspire or the other way around"))`
`39`	`44`	`}`