Merge pull request #6 from utopia-php/feat-throw-on-no-scheme

Feat throw on no scheme or host

composer.json

@@ -9,8 +9,8 @@
         "psr-4": {"Utopia\\DSN\\": "src/DSN"}
     },
     "scripts": {
-      "lint": "./vendor/bin/pint --test",
-      "format": "./vendor/bin/pint",
+      "lint": "./vendor/bin/pint --test --preset psr12",
+      "format": "./vendor/bin/pint --preset psr12",
       "test": "./vendor/bin/phpunit --configuration phpunit.xml tests"
     },
     "require": {

src/DSN/DSN.php

@@ -53,16 +53,24 @@ class DSN
      */
     public function __construct(string $dsn)
     {
-        $parts = parse_url($dsn);
+        $parts = \parse_url($dsn);
 
-        if (! $parts) {
+        if (!$parts) {
             throw new \InvalidArgumentException("Unable to parse DSN: $dsn");
         }
 
-        $this->scheme = $parts['scheme'] ?? null;
+        if (empty($parts['scheme'])) {
+            throw new \InvalidArgumentException('Unable to parse DSN: scheme is required');
+        }
+
+        if (empty($parts['host'])) {
+            throw new \InvalidArgumentException('Unable to parse DSN: host is required');
+        }
+
+        $this->scheme = $parts['scheme'];
         $this->user = isset($parts['user']) ? \urldecode($parts['user']) : null;
         $this->password = isset($parts['pass']) ? \urldecode($parts['pass']) : null;
-        $this->host = $parts['host'] ?? '';
+        $this->host = $parts['host'];
         $this->port = $parts['port'] ?? null;
         $this->path = isset($parts['path']) ? ltrim((string) $parts['path'], '/') : '';
         $this->query = $parts['query'] ?? null;