Make domain required in currentTunnelUrl

These methods are never passed null anyway.

Co-Authored-By: Mateus Junges <[email protected]>

Author: mattstauffer

cli/Valet/Cloudflared.php

@@ -10,7 +10,7 @@ public function __construct(public CommandLine $cli, public Brew $brew)
     {
     }
 
-    public function currentTunnelUrl(?string $domain = null)
+    public function currentTunnelUrl(string $domain): ?string
     {
         return $this->currentCloudflaredTunnels()[$domain] ?? false;
     }

cli/Valet/Expose.php

@@ -9,7 +9,7 @@ class Expose
 {
     public function __construct(public Composer $composer, public CommandLine $cli) {}
 
-    public function currentTunnelUrl(?string $domain = null): ?string
+    public function currentTunnelUrl(string $domain): ?string
     {
         $endpoint = 'http://127.0.0.1:4040/api/tunnels';
 

cli/Valet/Ngrok.php

@@ -18,7 +18,7 @@ public function __construct(public CommandLine $cli, public Brew $brew) {}
     /**
      * Get the current tunnel URL from the Ngrok API.
      */
-    public function currentTunnelUrl(?string $domain = null): string
+    public function currentTunnelUrl(string $domain): string
     {
         // wait a second for ngrok to start before attempting to find available tunnels
         sleep(1);