From c6ecece09c8d4a9e112db5a93e9392a02240a6fd Mon Sep 17 00:00:00 2001
From: MilesChou <jangconan@gmail.com>
Date: Fri, 25 Dec 2020 16:46:10 +0800
Subject: [PATCH] Fix OPTIONS method bug when use same path but diff domain

---
 .../Routing/CompiledRouteCollection.php       |  4 ++
 .../Routing/CompiledRouteCollectionTest.php   | 53 +++++++++++++++++++
 2 files changed, 57 insertions(+)

diff --git a/src/Illuminate/Routing/CompiledRouteCollection.php b/src/Illuminate/Routing/CompiledRouteCollection.php
index 099156cc4b81..c53954194e06 100644
--- a/src/Illuminate/Routing/CompiledRouteCollection.php
+++ b/src/Illuminate/Routing/CompiledRouteCollection.php
@@ -252,6 +252,10 @@ public function getRoutesByMethod()
             })
             ->map(function (Collection $routes) {
                 return $routes->mapWithKeys(function (Route $route) {
+                    if ($domain = $route->getDomain()) {
+                        return [$domain.'/'.$route->uri => $route];
+                    }
+
                     return [$route->uri => $route];
                 })->all();
             })
diff --git a/tests/Integration/Routing/CompiledRouteCollectionTest.php b/tests/Integration/Routing/CompiledRouteCollectionTest.php
index 22928a4c1f4f..e474e8a461c4 100644
--- a/tests/Integration/Routing/CompiledRouteCollectionTest.php
+++ b/tests/Integration/Routing/CompiledRouteCollectionTest.php
@@ -490,6 +490,59 @@ public function testTrailingSlashIsTrimmedWhenMatchingCachedRoutes()
         $this->assertSame('foo', $this->collection()->match($request)->getName());
     }
 
+    public function testRouteWithSamePathAndSameMethodButDiffDomainNameWithOptionsMethod()
+    {
+        $routes = [
+            'foo_domain' => $this->newRoute('GET', 'same/path', [
+                'uses' => 'FooController@index',
+                'as' => 'foo',
+                'domain' => 'foo.localhost',
+            ]),
+            'bar_domain' => $this->newRoute('GET', 'same/path', [
+                'uses' => 'BarController@index',
+                'as' => 'bar',
+                'domain' => 'bar.localhost',
+            ]),
+            'no_domain' => $this->newRoute('GET', 'same/path', [
+                'uses' => 'BarController@index',
+                'as' => 'no_domain',
+            ]),
+        ];
+
+        $this->routeCollection->add($routes['foo_domain']);
+        $this->routeCollection->add($routes['bar_domain']);
+        $this->routeCollection->add($routes['no_domain']);
+
+        $expectedMethods = [
+            'OPTIONS',
+        ];
+
+        $this->assertSame($expectedMethods, $this->collection()->match(
+            Request::create('http://foo.localhost/same/path', 'OPTIONS')
+        )->methods);
+
+        $this->assertSame($expectedMethods, $this->collection()->match(
+            Request::create('http://bar.localhost/same/path', 'OPTIONS')
+        )->methods);
+
+        $this->assertSame($expectedMethods, $this->collection()->match(
+            Request::create('http://no.localhost/same/path', 'OPTIONS')
+        )->methods);
+
+        $this->assertEquals([
+            'HEAD' => [
+                'foo.localhost/same/path' => $routes['foo_domain'],
+                'bar.localhost/same/path' => $routes['bar_domain'],
+                'same/path' => $routes['no_domain'],
+            ],
+            'GET' => [
+                'foo.localhost/same/path' => $routes['foo_domain'],
+                'bar.localhost/same/path' => $routes['bar_domain'],
+                'same/path' => $routes['no_domain'],
+            ],
+        ], $this->collection()->getRoutesByMethod());
+    }
+
     /**
      * Create a new Route object.
      *