Allow variable domain to use host as a fallback

Author: tanmaymishu

src/Storage/DatabaseRequestRepository.php

@@ -108,7 +108,9 @@ protected function routeResult(?array $route, ?array $responses)
             $route['description'],
             $route['content'],
             [
-                'domain' => $route['domain'],
+                'domain' => $this->hasVariableFragment($route['domain'])
+                    ? request()->getHost()
+                    : $route['domain'],
                 'methods' => $route['methods'],
                 'uri' => $route['uri'],
                 'name' => $route['name'],
@@ -144,4 +146,17 @@ protected function table($table)
     {
         return DB::connection($this->connection)->table($table);
     }
+
+    /**
+     * Determines if a given domain has any dynamic fragment.
+     *
+     * @param  string|null  $domain
+     * @return bool
+     */
+    private function hasVariableFragment(?string $domain)
+    {
+        return collect(explode('.', $domain))->filter(function ($fragment) {
+            return Str::startsWith($fragment, '{') && Str::endsWith($fragment,'}');
+        })->count() > 0;
+    }
 }

tests/Http/RoutesRequestTest.php

@@ -7,6 +7,7 @@
 use Davidhsianturi\Compass\Storage\RouteModel;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Davidhsianturi\Compass\Storage\DatabaseRequestRepository;
+use Illuminate\Support\Facades\URL;
 
 class RoutesRequestTest extends TestCase
 {
@@ -54,6 +55,19 @@ public function test_show_route_request_by_id()
             ->assertExactJson($route);
     }
 
+    public function test_show_route_request_fallbacks_to_host_for_variable_domain()
+    {
+        $host = 'test.tld.test';
+        URL::forceRootUrl('http://' . $host);
+        $this->registerVariableRoute();
+
+        $route = $this->repository->get()->first()->jsonSerialize();
+
+        $this->getJson(route('compass.request.show', $route['id']))
+            ->assertSuccessful()
+            ->assertJsonFragment(['domain' => $host]);
+    }
+
     public function test_store_the_route_request_to_storage()
     {
         $this->registerAppRoutes();

tests/TestCase.php

@@ -93,4 +93,15 @@ protected function registerAppRoutes()
             });
         });
     }
+
+    protected function registerVariableRoute()
+    {
+        RouteFacade::prefix('api/v1')->group(function () {
+            RouteFacade::group(['domain' => '{account}.tld.test'], function () {
+                RouteFacade::get('/test', function () {
+                    return 'test';
+                })->name('vardomain');
+            });
+        });
+    }
 }