add format-suffix handling

sulu · Aug 3, 2023 · 50acd25 · 50acd25
1 parent ea6267e
commit 50acd25
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 2 deletions.
diff --git a/Controller/WebsiteRedirectController.php b/Controller/WebsiteRedirectController.php
@@ -34,8 +34,12 @@ public function redirect(Request $request, RedirectRouteInterface $redirectRoute
 
         $queryString = http_build_query($request->query->all());
 
+        $requestFormat = $request->getRequestFormat(null);
+        $formatSuffix = $requestFormat ? ('.' . $requestFormat) : '';
+
         $url = [
             $redirectRoute->getTarget(),
+            $formatSuffix,
             false === strpos($redirectRoute->getTarget(), '?') ? '?' : '&',
             $queryString,
         ];

diff --git a/Routing/RedirectRouteProvider.php b/Routing/RedirectRouteProvider.php
@@ -49,10 +49,11 @@ public function getRouteCollectionForRequest(Request $request): RouteCollection
         // server encodes the url and symfony does not encode it
         // symfony decodes this data here https://github.com/symfony/symfony/blob/v5.2.3/src/Symfony/Component/Routing/Matcher/UrlMatcher.php#L88
         $pathInfo = rawurldecode($request->getPathInfo());
+        $path = \str_replace('.' . $request->getRequestFormat(), '', $pathInfo);
         $host = $request->getHost();
 
         $routeCollection = new RouteCollection();
-        if (!$redirectRoute = $this->redirectRouteRepository->findEnabledBySource($pathInfo, $host)) {
+        if (!$redirectRoute = $this->redirectRouteRepository->findEnabledBySource($path, $host)) {
             return $routeCollection;
         }
 

diff --git a/Tests/Functional/Routing/RedirectRouteProviderTest.php b/Tests/Functional/Routing/RedirectRouteProviderTest.php
@@ -38,6 +38,7 @@ public function testRoute(
         string $source,
         int $statusCode,
         string $target = '',
+        string $targetUrl = '',
         ?string $sourceHost = null
     ) {
         // setup models
@@ -61,7 +62,7 @@ public function testRoute(
 
         if ($target) {
             $this->assertInstanceOf(RedirectResponse::class, $response);
-            $this->assertSame($target, $response->getTargetUrl());
+            $this->assertSame($targetUrl, $response->getTargetUrl());
         }
     }
 
@@ -72,13 +73,15 @@ public function routeDataProvider(): \Generator
             '/test-301',
             301,
             '/test2',
+            '/test2',
         ];
 
         yield [
             '/test-302',
             '/test-302',
             302,
             '/test2',
+            '/test2',
         ];
 
         yield [
@@ -92,6 +95,7 @@ public function routeDataProvider(): \Generator
             '/test-domain-redirect',
             301,
             '/',
+            '/',
             'with-domain.com',
         ];
 
@@ -100,6 +104,15 @@ public function routeDataProvider(): \Generator
             '/test-emoticon-🎉',
             301,
             '/',
+            '/',
+        ];
+
+        yield [
+            '/source.json', // browsers will encode the url and be provided this way to symfony getPathInfo
+            '/source',
+            301,
+            '/target',
+            '/target.json',
         ];
     }
 }