Merge pull request #32 from PDOK/wr/error-format

small improvement

Author: Shalucik

pkg/validation/ingress_route_urls.go

@@ -22,7 +22,7 @@ func ValidateIngressRouteURLsContainsBaseURL(urls model.IngressRouteURLs, baseUR
 		}
 	}
 
-	return field.Invalid(path, urls, fmt.Sprintf("must contain baseURL: %s", baseURL))
+	return field.Invalid(path, fmt.Sprint(urls), fmt.Sprintf("must contain baseURL: %s", baseURL))
 }
 
 func ValidateIngressRouteURLsNotRemoved(oldURLs, newURLs model.IngressRouteURLs, allErrs *field.ErrorList, path *field.Path) {
@@ -42,7 +42,7 @@ func ValidateIngressRouteURLsNotRemoved(oldURLs, newURLs model.IngressRouteURLs,
 		}
 
 		if !found {
-			*allErrs = append(*allErrs, field.Invalid(path, newURLs, fmt.Sprintf("urls cannot be removed: %s", url)))
+			*allErrs = append(*allErrs, field.Invalid(path, fmt.Sprint(newURLs), fmt.Sprintf("urls cannot be removed, missing: %s", url)))
 		}
 	}
 }

pkg/validation/labels.go

@@ -23,7 +23,7 @@ func ValidateLabelsOnUpdate(oldLabels, newLabels map[string]string, allErrs *fie
 		if !ok {
 			*allErrs = append(*allErrs, field.Required(fieldPath.Child(oldKey), "labels cannot be removed"))
 		} else if oldValue != newValue {
-			*allErrs = append(*allErrs, field.Invalid(fieldPath.Child(oldKey), newValue, "immutable: should be "+oldValue))
+			*allErrs = append(*allErrs, field.Invalid(fieldPath.Child(oldKey), newValue, "immutable: should be: "+oldValue))
 		}
 	}