feat: multiple hostname routing

system/Router/RouteCollection.php

@@ -1562,7 +1562,7 @@ 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
+ * @param list<string>|string $hostname Hostname in route options
  */
  private function checkHostname($hostname): bool
  {
@@ -1571,6 +1571,13 @@ private function checkHostname($hostname): bool
  return false;
  }
 
+ // Has multiple hostname
+ if (is_array($hostname)) {
+ $hostnameLower = array_map('strtolower', $hostname);
+
+ return in_array(strtolower($this->httpHost), $hostnameLower, true);
+ }
+
  return strtolower($this->httpHost) === strtolower($hostname);
  }
 

tests/system/Router/RouteCollectionTest.php

@@ -13,6 +13,7 @@
 
 namespace CodeIgniter\Router;
 
+use App\Controllers\Home;
 use App\Controllers\Product;
 use CodeIgniter\Config\Services;
 use CodeIgniter\controller;
@@ -1744,6 +1745,46 @@ public function testRouteOverwritingMatchingHost(): void
  $this->assertSame($expects, $router->handle('/'));
  }
 
+ public function testRouteMatchingHostMultipleCorrect(): void
+ {
+ service('superglobals')->setServer('HTTP_HOST', 'two.domain.com');
+ service('request')->setMethod(Method::GET);
+
+ $routes = $this->getCollector();
+ $router = new Router($routes, Services::request());
+
+ $routes->setDefaultNamespace('App\Controllers');
+ $routes->setDefaultController('Home');
+ $routes->setDefaultMethod('index');
+
+ $routes->get('/', 'Home::index', ['as' => 'ddd']);
+ $routes->get('/', '\App\Controllers\Site\CDoc::index', ['hostname' => ['one.domain.com', 'two.domain.com', 'three.domain.com']]);
+
+ $expects = '\App\Controllers\Site\CDoc';
+
+ $this->assertSame($expects, $router->handle('/'));
+ }
+
+ public function testRouteMatchingHostMultipleFail(): void
+ {
+ service('superglobals')->setServer('HTTP_HOST', 'doc.domain.com');
+ service('request')->setMethod(Method::GET);
+
+ $routes = $this->getCollector();
+ $router = new Router($routes, Services::request());
+
+ $routes->setDefaultNamespace('App\Controllers');
+ $routes->setDefaultController('Home');
+ $routes->setDefaultMethod('index');
+
+ $routes->get('/', 'Home::index', ['as' => 'ddd']);
+ $routes->get('/', '\App\Controllers\Site\CDoc::index', ['hostname' => ['one.domain.com', 'two.domain.com', 'three.domain.com']]);
+
+ $expects = '\\' . Home::class;
+
+ $this->assertSame($expects, $router->handle('/'));
+ }
+
  /**
  * Tests for router DefaultNameSpace issue
  *

user_guide_src/source/changelogs/v4.6.0.rst

@@ -165,6 +165,11 @@ Commands
  arguments.
 - The ``spark filter:check`` command now displays filter classnames.
 
+Routing
+=======
+
+- Restrict limit hostname can multiple.
+
 Testing
 =======
 

user_guide_src/source/incoming/routing.rst

@@ -476,6 +476,12 @@ by passing the "hostname" option along with the desired domain to allow it on as
 This example would only allow the specified hosts to work if the domain exactly matched **accounts.example.com**.
 It would not work under the main site at **example.com**.
 
+.. versionadded:: 4.6.0
+
+Also you can restrict with multiple hostname, e.g:
+
+.. literalinclude:: routing/073.php
+
 Limit to Subdomains
 ===================
 

user_guide_src/source/incoming/routing/073.php

@@ -0,0 +1,3 @@
+<?php
+
+$routes->get('from', 'to', ['hostname' => ['s1.example.com', 's2.example.com']]);