Fixed: FS#1616 - Extended URL validation

tbrehm · tbrehm · commit ced7fc44a7b6 · 2011-09-05T14:15:53.000Z
diff --git a/interface/lib/classes/validate_cron.inc.php b/interface/lib/classes/validate_cron.inc.php
@@ -52,7 +52,7 @@ function command_format($field_name, $field_value, $validator) {
             
             if($parsed["scheme"] != "http" && $parsed["scheme"] != "https") return $this->get_error($validator['errmsg']);
             
-            if(preg_match("'^([a-z0-9][a-z0-9-]{0,62}\.)+([a-z]{2,4})$'i", $parsed["host"]) == false) return $this->get_error($validator['errmsg']);
+            if(preg_match("'^([a-z0-9][a-z0-9-]{0,62}\.)+([a-z]{2,30})$'i", $parsed["host"]) == false) return $this->get_error($validator['errmsg']);
         }
     }
     

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ function command_format($field_name, $field_value, $validator) {`
`52`	`52`
`53`	`53`	`if($parsed["scheme"] != "http" && $parsed["scheme"] != "https") return $this->get_error($validator['errmsg']);`
`54`	`54`
`55`		`- if(preg_match("'^([a-z0-9][a-z0-9-]{0,62}\.)+([a-z]{2,4})$'i", $parsed["host"]) == false) return $this->get_error($validator['errmsg']);`
	`55`	`+ if(preg_match("'^([a-z0-9][a-z0-9-]{0,62}\.)+([a-z]{2,30})$'i", $parsed["host"]) == false) return $this->get_error($validator['errmsg']);`
`56`	`56`	`}`
`57`	`57`	`}`
`58`	`58`