From a04c1fafd37636e0e6a8f55fc2393f243f242293 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christian=20L=C3=BCck?= <christian@clue.engineering>
Date: Sun, 14 Feb 2021 16:24:47 +0100
Subject: [PATCH 1/4] Refactor to move redirects to new `RedirectHandler`

---
 src/App.php                     | 14 ++----------
 src/FilesystemHandler.php       | 14 ++----------
 src/RedirectHandler.php         | 33 +++++++++++++++++++++++++++
 tests/AppTest.php               |  6 +++--
 tests/FilesystemHandlerTest.php |  2 ++
 tests/RedirectHandlerTest.php   | 40 +++++++++++++++++++++++++++++++++
 6 files changed, 83 insertions(+), 26 deletions(-)
 create mode 100644 src/RedirectHandler.php
 create mode 100644 tests/RedirectHandlerTest.php

diff --git a/src/App.php b/src/App.php
index 7b9636d..4b73dd4 100644
--- a/src/App.php
+++ b/src/App.php
@@ -7,7 +7,6 @@
 use React\EventLoop\Loop;
 use React\EventLoop\LoopInterface;
 use React\Http\HttpServer;
-use React\Http\Message\Response;
 use React\Http\Message\ServerRequest;
 use React\Promise\Deferred;
 use React\Promise\PromiseInterface;
@@ -113,18 +112,9 @@ public function map(array $methods, string $route, callable $handler, callable .
         $this->router->map($methods, $route, $handler, ...$handlers);
     }
 
-    public function redirect($route, $target, $code = 302)
+    public function redirect(string $route, string $target, int $code = 302): void
     {
-        return $this->get($route, function () use ($target, $code) {
-            return new Response(
-                $code,
-                [
-                    'Content-Type' => 'text/html',
-                    'Location' => $target
-                ],
-                'See ' . $target . '...' . "\n"
-            );
-        });
+        $this->get($route, new RedirectHandler($target, $code));
     }
 
     public function run()
diff --git a/src/FilesystemHandler.php b/src/FilesystemHandler.php
index cdd0f7b..acc2642 100644
--- a/src/FilesystemHandler.php
+++ b/src/FilesystemHandler.php
@@ -68,12 +68,7 @@ public function __invoke(ServerRequestInterface $request)
         \clearstatcache();
         if ($valid && \is_dir($path)) {
             if ($local !== '' && \substr($local, -1) !== '/') {
-                return new Response(
-                    302,
-                    [
-                        'Location' => \basename($path) . '/'
-                    ]
-                );
+                return (new RedirectHandler(\basename($path) . '/'))();
             }
 
             $response = '<strong>' . $this->html->escape($local === '' ? '/' : $local) . '</strong>' . "\n<ul>\n";
@@ -102,12 +97,7 @@ public function __invoke(ServerRequestInterface $request)
             );
         } elseif ($valid && \is_file($path)) {
             if ($local !== '' && \substr($local, -1) === '/') {
-                return new Response(
-                    302,
-                    [
-                        'Location' => '../' . \basename($path)
-                    ]
-                );
+                return (new RedirectHandler('../' . \basename($path)))();
             }
 
             // Assign MIME type based on file extension (same as nginx/Apache) or fall back to given default otherwise.
diff --git a/src/RedirectHandler.php b/src/RedirectHandler.php
new file mode 100644
index 0000000..af783c9
--- /dev/null
+++ b/src/RedirectHandler.php
@@ -0,0 +1,33 @@
+<?php
+
+namespace FrameworkX;
+
+use Psr\Http\Message\ResponseInterface;
+use React\Http\Message\Response;
+
+/**
+ * @internal
+ */
+class RedirectHandler
+{
+    private $target;
+    private $code;
+
+    public function __construct(string $target, int $code = 302)
+    {
+        $this->target = $target;
+        $this->code = $code;
+    }
+
+    public function __invoke(): ResponseInterface
+    {
+        return new Response(
+            $this->code,
+            [
+                'Content-Type' => 'text/html',
+                'Location' => $this->target
+            ],
+            'See ' . $this->target . '...' . "\n"
+        );
+    }
+}
diff --git a/tests/AppTest.php b/tests/AppTest.php
index f2beb6e..8659dfe 100644
--- a/tests/AppTest.php
+++ b/tests/AppTest.php
@@ -305,8 +305,9 @@ public function testRedirectMethodAddsGetRouteOnRouterWhichWhenInvokedReturnsRed
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
-        $this->assertEquals(302, $response->getStatusCode());
         $this->assertEquals('text/html', $response->getHeaderLine('Content-Type'));
+
+        $this->assertEquals(302, $response->getStatusCode());
         $this->assertEquals('/users', $response->getHeaderLine('Location'));
         $this->assertEquals("See /users...\n", (string) $response->getBody());
     }
@@ -334,8 +335,9 @@ public function testRedirectMethodWithCustomRedirectCodeAddsGetRouteOnRouterWhic
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
-        $this->assertEquals(307, $response->getStatusCode());
         $this->assertEquals('text/html', $response->getHeaderLine('Content-Type'));
+
+        $this->assertEquals(307, $response->getStatusCode());
         $this->assertEquals('/users', $response->getHeaderLine('Location'));
         $this->assertEquals("See /users...\n", (string) $response->getBody());
     }
diff --git a/tests/FilesystemHandlerTest.php b/tests/FilesystemHandlerTest.php
index 9a5acd2..8ed06ee 100644
--- a/tests/FilesystemHandlerTest.php
+++ b/tests/FilesystemHandlerTest.php
@@ -267,6 +267,7 @@ public function testInvokeWithValidPathToDirectoryButWithoutTrailingSlashWillRet
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
+
         $this->assertEquals(302, $response->getStatusCode());
         $this->assertEquals('.github/', $response->getHeaderLine('Location'));
     }
@@ -282,6 +283,7 @@ public function testInvokeWithValidPathToFileButWithTrailingSlashWillReturnRedir
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
+
         $this->assertEquals(302, $response->getStatusCode());
         $this->assertEquals('../LICENSE', $response->getHeaderLine('Location'));
     }
diff --git a/tests/RedirectHandlerTest.php b/tests/RedirectHandlerTest.php
new file mode 100644
index 0000000..685f3f7
--- /dev/null
+++ b/tests/RedirectHandlerTest.php
@@ -0,0 +1,40 @@
+<?php
+
+namespace Framework\Tests;
+
+use FrameworkX\RedirectHandler;
+use PHPUnit\Framework\TestCase;
+use Psr\Http\Message\ResponseInterface;
+
+class RedirectHandlerTest extends TestCase
+{
+    public function testInvokeReturnsResponseWithRedirectToGivenLocation()
+    {
+        $handler = new RedirectHandler('http://example.com/');
+
+        $response = $handler();
+
+        /** @var ResponseInterface $response */
+        $this->assertInstanceOf(ResponseInterface::class, $response);
+        $this->assertEquals('text/html', $response->getHeaderLine('Content-Type'));
+
+        $this->assertEquals(302, $response->getStatusCode());
+        $this->assertEquals('http://example.com/', $response->getHeaderLine('Location'));
+        $this->assertEquals("See http://example.com/...\n", (string ) $response->getBody());
+    }
+
+    public function testInvokeReturnsResponseWithPermanentRedirectToGivenLocationAndCode()
+    {
+        $handler = new RedirectHandler('/', 301);
+
+        $response = $handler();
+
+        /** @var ResponseInterface $response */
+        $this->assertInstanceOf(ResponseInterface::class, $response);
+        $this->assertEquals('text/html', $response->getHeaderLine('Content-Type'));
+
+        $this->assertEquals(301, $response->getStatusCode());
+        $this->assertEquals('/', $response->getHeaderLine('Location'));
+        $this->assertEquals("See /...\n", (string ) $response->getBody());
+    }
+}

From f1f85ec731e0810c1debefa340b69714de4b21cc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christian=20L=C3=BCck?= <christian@clue.engineering>
Date: Fri, 17 Sep 2021 10:11:55 +0200
Subject: [PATCH 2/4] Improve HTML output of redirection page (not usually
 displayed)

---
 src/HtmlHandler.php             |  1 +
 src/RedirectHandler.php         | 28 +++++++++-----
 tests/AppTest.php               | 12 ++++--
 tests/FilesystemHandlerTest.php |  8 ++++
 tests/RedirectHandlerTest.php   | 68 +++++++++++++++++++++++++++++++--
 tests/acceptance.sh             |  6 +--
 6 files changed, 103 insertions(+), 20 deletions(-)

diff --git a/src/HtmlHandler.php b/src/HtmlHandler.php
index a81d666..81e277b 100644
--- a/src/HtmlHandler.php
+++ b/src/HtmlHandler.php
@@ -25,6 +25,7 @@ public function statusResponse(int $statusCode, string $title, string $subtitle,
 strong { color: #111827; font-size: 3em; }
 p { margin: .5em 0 0 0; grid-column: 2; color: #6b7280; }
 code { padding: 0 .3em; background-color: #f5f6f9; }
+a { color: inherit; }
 </style>
 </head>
 <body>
diff --git a/src/RedirectHandler.php b/src/RedirectHandler.php
index af783c9..80a9875 100644
--- a/src/RedirectHandler.php
+++ b/src/RedirectHandler.php
@@ -12,22 +12,32 @@ class RedirectHandler
 {
     private $target;
     private $code;
+    private $reason;
 
-    public function __construct(string $target, int $code = 302)
+    /** @var HtmlHandler */
+    private $html;
+
+    public function __construct(string $target, int $redirectStatusCode = 302)
     {
+        if ($redirectStatusCode < 300 || $redirectStatusCode === 304 || $redirectStatusCode >= 400) {
+            throw new \InvalidArgumentException('Invalid redirect status code given');
+        }
+
         $this->target = $target;
-        $this->code = $code;
+        $this->code = $redirectStatusCode;
+        $this->reason = \ucwords((new Response($redirectStatusCode))->getReasonPhrase()) ?: 'Redirect';
+        $this->html = new HtmlHandler();
     }
 
     public function __invoke(): ResponseInterface
     {
-        return new Response(
+        $url = $this->html->escape($this->target);
+
+        return $this->html->statusResponse(
             $this->code,
-            [
-                'Content-Type' => 'text/html',
-                'Location' => $this->target
-            ],
-            'See ' . $this->target . '...' . "\n"
-        );
+            'Redirecting to ' . $url,
+            $this->reason,
+            "<p>Redirecting to <a href=\"$url\"><code>$url</code></a>...</p>\n"
+        )->withHeader('Location', $this->target);
     }
 }
diff --git a/tests/AppTest.php b/tests/AppTest.php
index 8659dfe..2bc6410 100644
--- a/tests/AppTest.php
+++ b/tests/AppTest.php
@@ -305,11 +305,13 @@ public function testRedirectMethodAddsGetRouteOnRouterWhichWhenInvokedReturnsRed
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
-        $this->assertEquals('text/html', $response->getHeaderLine('Content-Type'));
+        $this->assertEquals('text/html; charset=utf-8', $response->getHeaderLine('Content-Type'));
+        $this->assertStringMatchesFormat("<!DOCTYPE html>\n<html>%a</html>\n", (string) $response->getBody());
 
         $this->assertEquals(302, $response->getStatusCode());
         $this->assertEquals('/users', $response->getHeaderLine('Location'));
-        $this->assertEquals("See /users...\n", (string) $response->getBody());
+        $this->assertStringContainsString("<title>Redirecting to /users</title>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<p>Redirecting to <a href=\"/users\"><code>/users</code></a>...</p>\n", (string) $response->getBody());
     }
 
     public function testRedirectMethodWithCustomRedirectCodeAddsGetRouteOnRouterWhichWhenInvokedReturnsRedirectResponseWithCustomRedirectCode()
@@ -335,11 +337,13 @@ public function testRedirectMethodWithCustomRedirectCodeAddsGetRouteOnRouterWhic
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
-        $this->assertEquals('text/html', $response->getHeaderLine('Content-Type'));
+        $this->assertEquals('text/html; charset=utf-8', $response->getHeaderLine('Content-Type'));
+        $this->assertStringMatchesFormat("<!DOCTYPE html>\n<html>%a</html>\n", (string) $response->getBody());
 
         $this->assertEquals(307, $response->getStatusCode());
         $this->assertEquals('/users', $response->getHeaderLine('Location'));
-        $this->assertEquals("See /users...\n", (string) $response->getBody());
+        $this->assertStringContainsString("<title>Redirecting to /users</title>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<p>Redirecting to <a href=\"/users\"><code>/users</code></a>...</p>\n", (string) $response->getBody());
     }
 
     public function testRequestFromGlobalsWithNoServerVariablesDefaultsToGetRequestToLocalhost()
diff --git a/tests/FilesystemHandlerTest.php b/tests/FilesystemHandlerTest.php
index 8ed06ee..6fe898a 100644
--- a/tests/FilesystemHandlerTest.php
+++ b/tests/FilesystemHandlerTest.php
@@ -267,9 +267,13 @@ public function testInvokeWithValidPathToDirectoryButWithoutTrailingSlashWillRet
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
+        $this->assertEquals('text/html; charset=utf-8', $response->getHeaderLine('Content-Type'));
+        $this->assertStringMatchesFormat("<!DOCTYPE html>\n<html>%a</html>\n", (string) $response->getBody());
 
         $this->assertEquals(302, $response->getStatusCode());
         $this->assertEquals('.github/', $response->getHeaderLine('Location'));
+        $this->assertStringContainsString("<title>Redirecting to .github/</title>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<p>Redirecting to <a href=\".github/\"><code>.github/</code></a>...</p>\n", (string) $response->getBody());
     }
 
     public function testInvokeWithValidPathToFileButWithTrailingSlashWillReturnRedirectToPathWithoutSlash()
@@ -283,8 +287,12 @@ public function testInvokeWithValidPathToFileButWithTrailingSlashWillReturnRedir
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
+        $this->assertEquals('text/html; charset=utf-8', $response->getHeaderLine('Content-Type'));
+        $this->assertStringMatchesFormat("<!DOCTYPE html>\n<html>%a</html>\n", (string) $response->getBody());
 
         $this->assertEquals(302, $response->getStatusCode());
         $this->assertEquals('../LICENSE', $response->getHeaderLine('Location'));
+        $this->assertStringContainsString("<title>Redirecting to ../LICENSE</title>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<p>Redirecting to <a href=\"../LICENSE\"><code>../LICENSE</code></a>...</p>\n", (string) $response->getBody());
     }
 }
diff --git a/tests/RedirectHandlerTest.php b/tests/RedirectHandlerTest.php
index 685f3f7..1a111c6 100644
--- a/tests/RedirectHandlerTest.php
+++ b/tests/RedirectHandlerTest.php
@@ -16,11 +16,17 @@ public function testInvokeReturnsResponseWithRedirectToGivenLocation()
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
-        $this->assertEquals('text/html', $response->getHeaderLine('Content-Type'));
+        $this->assertEquals('text/html; charset=utf-8', $response->getHeaderLine('Content-Type'));
+        $this->assertStringMatchesFormat("<!DOCTYPE html>\n<html>%a</html>\n", (string) $response->getBody());
+        $this->assertStringMatchesFormat("%a<style nonce=\"%s\">\n%a", (string) $response->getBody());
+        $this->assertStringMatchesFormat('style-src \'nonce-%s\'; img-src \'self\'; default-src \'none\'', $response->getHeaderLine('Content-Security-Policy'));
 
         $this->assertEquals(302, $response->getStatusCode());
         $this->assertEquals('http://example.com/', $response->getHeaderLine('Location'));
-        $this->assertEquals("See http://example.com/...\n", (string ) $response->getBody());
+        $this->assertStringContainsString("<title>Redirecting to http://example.com/</title>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<h1>302</h1>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<strong>Found</strong>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<p>Redirecting to <a href=\"http://example.com/\"><code>http://example.com/</code></a>...</p>\n", (string) $response->getBody());
     }
 
     public function testInvokeReturnsResponseWithPermanentRedirectToGivenLocationAndCode()
@@ -31,10 +37,64 @@ public function testInvokeReturnsResponseWithPermanentRedirectToGivenLocationAnd
 
         /** @var ResponseInterface $response */
         $this->assertInstanceOf(ResponseInterface::class, $response);
-        $this->assertEquals('text/html', $response->getHeaderLine('Content-Type'));
 
         $this->assertEquals(301, $response->getStatusCode());
         $this->assertEquals('/', $response->getHeaderLine('Location'));
-        $this->assertEquals("See /...\n", (string ) $response->getBody());
+        $this->assertStringContainsString("<title>Redirecting to /</title>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<h1>301</h1>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<strong>Moved Permanently</strong>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<p>Redirecting to <a href=\"/\"><code>/</code></a>...</p>\n", (string) $response->getBody());
+    }
+
+    public function testInvokeReturnsResponseWithCustomRedirectStatusCodeAndGivenLocation()
+    {
+        $handler = new RedirectHandler('/', 399);
+
+        $response = $handler();
+
+        /** @var ResponseInterface $response */
+        $this->assertInstanceOf(ResponseInterface::class, $response);
+
+        $this->assertEquals(399, $response->getStatusCode());
+        $this->assertEquals('/', $response->getHeaderLine('Location'));
+        $this->assertStringContainsString("<title>Redirecting to /</title>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<h1>399</h1>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<strong>Redirect</strong>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<p>Redirecting to <a href=\"/\"><code>/</code></a>...</p>\n", (string) $response->getBody());
+    }
+
+    public function testInvokeReturnsResponseWithRedirectToGivenLocationWithSpecialCharsEncoded()
+    {
+        $handler = new RedirectHandler('/hello%20w%7Frld?a=1&b=2');
+
+        $response = $handler();
+
+        /** @var ResponseInterface $response */
+        $this->assertInstanceOf(ResponseInterface::class, $response);
+
+        $this->assertEquals(302, $response->getStatusCode());
+        $this->assertEquals('/hello%20w%7Frld?a=1&b=2', $response->getHeaderLine('Location'));
+        $this->assertStringContainsString("<title>Redirecting to /hello%20w%7Frld?a=1&amp;b=2</title>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<h1>302</h1>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<strong>Found</strong>\n", (string) $response->getBody());
+        $this->assertStringContainsString("<p>Redirecting to <a href=\"/hello%20w%7Frld?a=1&amp;b=2\"><code>/hello%20w%7Frld?a=1&amp;b=2</code></a>...</p>\n", (string) $response->getBody());
+    }
+
+    public function testConstructWithSuccessCodeThrows()
+    {
+        $this->expectException(\InvalidArgumentException::class);
+        new RedirectHandler('/', 200);
+    }
+
+    public function testConstructWithNotModifiedCodeThrows()
+    {
+        $this->expectException(\InvalidArgumentException::class);
+        new RedirectHandler('/', 304);
+    }
+
+    public function testConstructWithBadRequestCodeThrows()
+    {
+        $this->expectException(\InvalidArgumentException::class);
+        new RedirectHandler('/', 400);
     }
 }
diff --git a/tests/acceptance.sh b/tests/acceptance.sh
index 5b230aa..c823e02 100755
--- a/tests/acceptance.sh
+++ b/tests/acceptance.sh
@@ -71,14 +71,14 @@ out=$(curl -v $base/users/ 2>&1);    match "HTTP/.* 404"
 out=$(curl -v $base/users/a/b 2>&1); match "HTTP/.* 404"
 
 out=$(curl -v $base/LICENSE 2>&1);              match "HTTP/.* 200" && match -iP "Content-Type: text/plain[\r\n]"
-out=$(curl -v $base/source 2>&1);               match -i "Location: /source/" && match -iP "Content-Type: text/html[\r\n]"
+out=$(curl -v $base/source 2>&1);               match -i "Location: /source/" && match -iP "Content-Type: text/html; charset=utf-8[\r\n]"
 out=$(curl -v $base/source/ 2>&1);              match "HTTP/.* 200"
 out=$(curl -v $base/source/composer.json 2>&1); match "HTTP/.* 200" && match -iP "Content-Type: application/json[\r\n]"
 out=$(curl -v $base/source/LICENSE 2>&1);       match "HTTP/.* 200" && match -iP "Content-Type: text/plain[\r\n]"
-out=$(curl -v $base/source/LICENSE/ 2>&1);      match -i "Location: ../LICENSE"
+out=$(curl -v $base/source/LICENSE/ 2>&1);      match -i "Location: ../LICENSE" && match -iP "Content-Type: text/html; charset=utf-8[\r\n]"
 out=$(curl -v $base/source/LICENSE// 2>&1);     match "HTTP/.* 404"
 out=$(curl -v $base/source//LICENSE 2>&1);      match "HTTP/.* 404"
-out=$(curl -v $base/source/tests 2>&1);         match -i "Location: tests/"
+out=$(curl -v $base/source/tests 2>&1);         match -i "Location: tests/" && match -iP "Content-Type: text/html; charset=utf-8[\r\n]"
 out=$(curl -v $base/source/invalid 2>&1);       match "HTTP/.* 404"
 out=$(curl -v $base/source/bin%00ary 2>&1);     match "HTTP/.* 40[40]" # expects 404, but not processed with nginx (400) and Apache (404)
 

From abf6562105ab0ed5c64b1b3107def77dfb33aef6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christian=20L=C3=BCck?= <christian@clue.engineering>
Date: Sun, 19 Sep 2021 12:29:22 +0200
Subject: [PATCH 3/4] Register redirects for all request methods

---
 src/App.php       | 2 +-
 tests/AppTest.php | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/App.php b/src/App.php
index 4b73dd4..00519db 100644
--- a/src/App.php
+++ b/src/App.php
@@ -114,7 +114,7 @@ public function map(array $methods, string $route, callable $handler, callable .
 
     public function redirect(string $route, string $target, int $code = 302): void
     {
-        $this->get($route, new RedirectHandler($target, $code));
+        $this->any($route, new RedirectHandler($target, $code));
     }
 
     public function run()
diff --git a/tests/AppTest.php b/tests/AppTest.php
index 2bc6410..78fbe58 100644
--- a/tests/AppTest.php
+++ b/tests/AppTest.php
@@ -282,13 +282,13 @@ public function testMapMethodAddsRouteOnRouter()
         $app->map(['GET', 'POST'], '/', function () { });
     }
 
-    public function testRedirectMethodAddsGetRouteOnRouterWhichWhenInvokedReturnsRedirectResponseWithTargetLocation()
+    public function testRedirectMethodAddsAnyRouteOnRouterWhichWhenInvokedReturnsRedirectResponseWithTargetLocation()
     {
         $app = new App();
 
         $handler = null;
         $router = $this->createMock(RouteHandler::class);
-        $router->expects($this->once())->method('map')->with(['GET'], '/', $this->callback(function ($fn) use (&$handler) {
+        $router->expects($this->once())->method('map')->with(['GET', 'POST', 'PUT', 'PATCH', 'DELETE', 'OPTIONS'], '/', $this->callback(function ($fn) use (&$handler) {
             $handler = $fn;
             return true;
         }));
@@ -314,13 +314,13 @@ public function testRedirectMethodAddsGetRouteOnRouterWhichWhenInvokedReturnsRed
         $this->assertStringContainsString("<p>Redirecting to <a href=\"/users\"><code>/users</code></a>...</p>\n", (string) $response->getBody());
     }
 
-    public function testRedirectMethodWithCustomRedirectCodeAddsGetRouteOnRouterWhichWhenInvokedReturnsRedirectResponseWithCustomRedirectCode()
+    public function testRedirectMethodWithCustomRedirectCodeAddsAnyRouteOnRouterWhichWhenInvokedReturnsRedirectResponseWithCustomRedirectCode()
     {
         $app = new App();
 
         $handler = null;
         $router = $this->createMock(RouteHandler::class);
-        $router->expects($this->once())->method('map')->with(['GET'], '/', $this->callback(function ($fn) use (&$handler) {
+        $router->expects($this->once())->method('map')->with(['GET', 'POST', 'PUT', 'PATCH', 'DELETE', 'OPTIONS'], '/', $this->callback(function ($fn) use (&$handler) {
             $handler = $fn;
             return true;
         }));

From 8c7c40fc19e52fda3126ef4391336c99185746cb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christian=20L=C3=BCck?= <christian@clue.engineering>
Date: Sun, 19 Sep 2021 14:16:53 +0200
Subject: [PATCH 4/4] Documentation for redirects

---
 docs/api/app.md      | 27 ++++++++++++++++++++++++---
 docs/api/response.md | 14 +++++++-------
 2 files changed, 31 insertions(+), 10 deletions(-)

diff --git a/docs/api/app.md b/docs/api/app.md
index f881269..e18aa3a 100644
--- a/docs/api/app.md
+++ b/docs/api/app.md
@@ -40,9 +40,9 @@ which also highlights how you can use [request attributes](request.md#attributes
 to access values from URI templates.
 
 An HTTP `GET` request for `/foo` would automatically reject the HTTP request with
-a 404 (Not Found) error response unless this route is registered.
-Likewise, an HTTP `POST` request for `/user` would reject with a 405 (Method Not
-Allowed) error response unless a route for this method is also registered.
+a `404 Not Found` error response unless this route is registered.
+Likewise, an HTTP `POST` request for `/user` would reject with a `405 Method Not
+Allowed` error response unless a route for this method is also registered.
 
 You can route any number of incoming HTTP requests to controller functions by
 using the matching API methods like this:
@@ -71,6 +71,27 @@ you can use this additional shortcut:
 $app->any('/user/{id}', $controller);
 ```
 
+## Redirects
+
+The `App` also offers a convenient helper method to redirect a matching route to
+a new URL like this:
+
+```php
+$app->redirect('/promo/reactphp', 'http://reactphp.org/');
+```
+
+Browsers and search engine crawlers will automatically follow the redirect with
+the `302 Found` status code by default. You can optionally pass a custom redirect
+status code in the `3xx` range to use. If this is a permanent redirect, you may
+want to use the `301 Permanent Redirect` status code to instruct search engine
+crawlers to update their index like this:
+
+```php
+$app->redirect('/blog.html', '/blog', 301);
+```
+
+See [response status codes](response.md#status-codes) for more details.
+
 ## Controllers
 
 The above examples use inline closures as controller functions to make these
diff --git a/docs/api/response.md b/docs/api/response.md
index 7010fad..d5c530a 100644
--- a/docs/api/response.md
+++ b/docs/api/response.md
@@ -164,12 +164,12 @@ Each HTTP response message contains a status code that describes whether the
 HTTP request has been successfully completed.
 Here's a list of the most common HTTP status codes:
 
-* 200 (OK)
-* 301 (Permanent Redirect)
-* 302 (Temporary Redirect)
-* 403 (Forbidden)
-* 404 (Not Found)
-* 500 (Internal Server Error)
+* `200 OK`
+* `301 Permanent Redirect`
+* `302 Found` (previously `302 Temporary Redirect`)
+* `403 Forbidden`
+* `404 Not Found`
+* `500 Internal Server Error`
 * …
 
 See [list of HTTP status codes](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status) for more details.
@@ -210,7 +210,7 @@ know what you're doing.
 Each controller function needs to return a response object in order to send
 an HTTP response message.
 If the controller functions throws an `Exception` (or `Throwable`) or any other type, the
-HTTP request will automatically be rejected with a 500 (Internal Server Error)
+HTTP request will automatically be rejected with a `500 Internal Server Error`
 HTTP error response:
 
 ```php