Merge pull request #2 from domainvalidity/2.x

Validate that the host is valid

Author: adrorocker

src/Host/Host.php

@@ -11,9 +11,10 @@ public function __construct(
         public ?string $host = null,
         public ?string $domain = null,
         public ?string $tld = null,
+        private ?bool $isValid = null,
         public ?bool $isPrivate = null,
     ) {
-        $parsed = HostParser::parse($this->original);
+        $parsed = HostParser::parse(strtolower($this->original));
 
         $this->host = strval($parsed['host']);
     }
@@ -38,6 +39,11 @@ public function tld(?string $tld = null): string|self
         $this->tld = $tld;
 
         $root = str_replace(strval($this->tld), '', strval($this->host));
+
+        if (!validate_domain_root(trim($root, '.'))) {
+            $this->isValid = false;
+        }
+
         $root = explode('.', trim(strval($root), '.'));
 
         $this->domain(end($root) . '.' . $this->tld);
@@ -58,7 +64,7 @@ public function domain(?string $domain = null): string|self
 
     public function isValid(): bool
     {
-        return !empty($this->tld());
+        return $this->isValid ?? !empty($this->tld());
     }
 
     public function isPrivate(?bool $isPrivate = null): bool|self

src/functions.php

@@ -23,3 +23,9 @@ function remove_empty_lines(string $text): ?string
 
     return $text;
 }
+
+function validate_domain_root(string $root): bool
+{
+    // Check if the string only contains alphanumeric values, dots or dashes
+    return preg_match('/^[a-zA-Z0-9.-]+$/', $root) === 1;
+}

tests/Unit/ValidatorTest.php

@@ -52,3 +52,13 @@
     it('empty domain', fn () =>  expect($host->domain())->toBe(''));
     it("is same as $url", fn () =>  expect($host->original())->toBe($url));
 });
+
+describe('Invalid *.adro.com', function () {
+    $validator = getInstance();
+    $url = 'https://*.adro.com';
+    $host = $validator->validate($url);
+
+    it('is not a valid domain', fn () =>  expect($host->isValid())->toBeFalse());
+    it('is not empty domain', fn () =>  expect($host->domain())->toBe('adro.com'));
+    it("is same as $url", fn () =>  expect($host->original())->toBe($url));
+});