Update valiadation for annotation

Author: AlexFenlon

internal/configs/annotations.go

@@ -505,11 +505,7 @@ func parseAnnotations(ingEx *IngressEx, baseCfgParams *ConfigParams, isPlus bool
 	}
 
 	if appRoot, exists := ingEx.Ingress.Annotations["nginx.org/app-root"]; exists {
-		if !VerifyPath(appRoot) {
-			nl.Errorf(l, "Ingress %s/%s: Invalid value nginx.org/app-root: got %q. Must start with '/'", ingEx.Ingress.GetNamespace(), ingEx.Ingress.GetName(), appRoot)
-		} else {
-			cfgParams.AppRoot = appRoot
-		}
+		cfgParams.AppRoot = appRoot
 	}
 
 	if useClusterIP, exists, err := GetMapKeyAsBool(ingEx.Ingress.Annotations, UseClusterIPAnnotation, ingEx.Ingress); exists {

internal/k8s/validation.go

@@ -363,6 +363,7 @@ var (
 		},
 		appRootAnnotation: {
 			validateAppRootAnnotation,
+			validateRewriteTargetAnnotation,
 		},
 	}
 	annotationNames = sortedAnnotationNames(annotationValidations)
@@ -394,9 +395,9 @@ func validateAppRootAnnotation(context *annotationValidationContext) field.Error
 		return allErrs
 	}
 
-	if !configs.VerifyPath(path) {
-		allErrs = append(allErrs, field.Invalid(context.fieldPath, path, "path must start with '/' and must not include any special character, '{', '}', ';' or '$'"))
-		return allErrs
+	validPath := regexp.MustCompile(`^/[a-zA-Z0-9\-_./]*$`)
+	if !validPath.MatchString(path) {
+		allErrs = append(allErrs, field.Invalid(context.fieldPath, path, "contains invalid characters, only alphanumeric, hyphens, underscores, dots, and forward slashes are allowed"))
 	}
 
 	// Ensure path doesn't end with /