feat: allow single-label hostnames in ValidateURL and improve performance

pkg/api/instance_handler.go

@@ -71,7 +71,7 @@ func UpdateInstance(instanceName string, instance models.Instance) error {
 		return err
 	}
 
-	log.Infof("Instance %s updated", instance.Name)
+	fmt.Printf("Instance %s updated\n", instance.Name)
 	return nil
 }
 

pkg/utils/helper.go

@@ -177,10 +177,11 @@ func ValidateRegistryName(rn string) bool {
 	return re.MatchString(rn)
 }
 
-// ValidateURL checks if the URL has valid format, non-empty host, and host is a valid IP or domain.
-// Domain regex: labels must start/end with alphanumeric, can contain hyphens, max 63 chars, TLD min 2 letters.
+// ValidateURL checks that the URL has a valid format and a non-empty host.
+// The host may be an IP address, "localhost", or an RFC 1123-style hostname validated by domainNameRegex.
 func ValidateURL(rawURL string) error {
-	var domainNameRegex = regexp.MustCompile(`^(?:[a-zA-Z0-9](?:[a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?\.)+[a-zA-Z]{2,}$`)
+	pattern := `^(?i:[a-z0-9](?:[a-z0-9-]{0,61}[a-z0-9])?)(?:\.[a-z0-9](?:[a-z0-9-]{0,61}[a-z0-9])?)*$`
+	domainNameRegex := regexp.MustCompile(pattern)
 
 	parsedURL, err := url.ParseRequestURI(rawURL)
 	if err != nil {

pkg/utils/helper_test.go

@@ -147,6 +147,9 @@ func TestValidateURL(t *testing.T) {
 		{"valid localhost https", "https://localhost", false},
 		{"valid localhost http with port", "http://localhost:8080", false},
 		{"valid localhost https with port", "https://localhost:8443", false},
+		{"valid single-label hostname", "http://harbor", false},
+		{"valid single-label with hyphen", "https://registry-server", false},
+		{"valid single-label with port", "http://harbor:8080", false},
 
 		// invalid URLs
 		{"empty string", "", true},