Prevent caching on 302, add wildcard check

composer.json

@@ -19,6 +19,7 @@
     "php": "^7.2.0",
     "psr/http-message": "^1.0",
     "slim/psr7": "^1.1",
+    "slim/http-cache": "^1.0",
     "laminas/laminas-httphandlerrunner": "^1.2"
   },
   "require-dev": {

src/RedirectRule.php

@@ -94,6 +94,11 @@ public function setSource(string $source): self
         return $this;
     }
 
+    public function isWildcard(): bool
+    {
+        return (strpos($this->source, '*') !== false) ? true : false;
+    }
+
     public function toArray(): array
     {
         return \get_object_vars($this);

src/RedirectUri.php

@@ -5,6 +5,7 @@
 
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\UriInterface;
+use Slim\HttpCache\Cache;
 use Slim\Psr7\Uri;
 
 class RedirectUri extends Uri
@@ -55,6 +56,12 @@ public function toRedirectResponse(ResponseInterface $response = null): Response
     {
         $factory = new RedirectResponseFactory();
         $response = $factory->createRedirectResponse($this, $response);
+        if ($response->getStatusCode() == 302) {
+            $cacheProvider = new \Slim\HttpCache\CacheProvider();
+            $response = $cacheProvider->denyCache($response);
+            $response = $response->withHeader('Pragma', 'no-cache');
+            $response = $cacheProvider->withExpires($response, strtotime('Yesterday'));
+        }
         return $response;
     }
 }

tests/ControllerTest.php

@@ -359,6 +359,27 @@ public function testRedirectSimpleWildcardPath()
         $this->assertEquals('https://localhost/wildcard/test?query=string', $result->location);
     }
 
+    public function testNoCacheOn302()
+    {
+        // 'Cache-Control', 'no-store,no-cache'
+        // 'Pragma', 'no-cache'
+        // 'Expires', strtotime('Yesterday')
+        $rule = [
+            "id" => "1",
+            "source" => "/",
+            "type" => "path",
+            "destination" => "/root",
+            "httpStatus" => 302,
+            "active" => 1
+        ];
+        $result = $this->slimRedirect('https://localhost/?query=string', [$rule]);
+        $this->assertEquals($rule['httpStatus'], $result->responseStatus);
+        $this->assertEquals($result->location, 'https://localhost/root?query=string');
+        $this->assertEquals('no-store,no-cache', $result->response->getHeaderLine('Cache-Control'));
+        $this->assertEquals('no-cache', $result->response->getHeaderLine('Pragma'));
+        $this->assertGreaterThanOrEqual(strtotime('Yesterday'), strtotime($result->response->getHeaderLine('Expires')));
+    }
+
     public function testParseDestination()
     {
         $rule = [

tests/RedirectRuleTest.php

@@ -67,6 +67,20 @@ public function testCreationFactory()
         $this->assertInstanceOf(RedirectRule::class, $rule);
     }
 
+    public function testIsWildcard()
+    {
+        $rule = [
+            "id" => "1",
+            "source" => "/wild/*/card",
+            "type" => "path",
+            "destination" => "/wildcard/*",
+            "httpStatus" => 301,
+            "active" => 1
+        ];
+        $redirectRule = RedirectRule::factory($rule);
+        $this->assertTrue($redirectRule->isWildcard());
+    }
+
     public function testToArray()
     {
         $rule = [