Merge pull request #7176 from kenjis/fix-spark-routes-hostname

fix: `spark routes` shows incorrect hostname routes

Author: kenjis

system/Router/RouteCollection.php

@@ -1316,13 +1316,12 @@ protected function create(string $verb, string $from, $to, ?array $options = nul
         // Hostname limiting?
         if (! empty($options['hostname'])) {
             // @todo determine if there's a way to whitelist hosts?
-            if (isset($this->httpHost) && strtolower($this->httpHost) !== strtolower($options['hostname'])) {
+            if (! $this->checkHostname($options['hostname'])) {
                 return;
             }
 
             $overwrite = true;
         }
-
         // Limiting to subdomains?
         elseif (! empty($options['subdomain'])) {
             // If we don't match the current subdomain, then
@@ -1395,6 +1394,22 @@ protected function create(string $verb, string $from, $to, ?array $options = nul
         }
     }
 
+    /**
+     * Compares the hostname passed in against the current hostname
+     * on this page request.
+     *
+     * @param string $hostname Hostname in route options
+     */
+    private function checkHostname($hostname): bool
+    {
+        // CLI calls can't be on hostname.
+        if (! isset($this->httpHost)) {
+            return false;
+        }
+
+        return strtolower($this->httpHost) === strtolower($hostname);
+    }
+
     private function processArrayCallableSyntax(string $from, array $to): string
     {
         // [classname, method]