Always call ->send() after apply header Location

samsonasik · Jan 18, 2017 · 3ad900f · 3ad900f
1 parent f67deab
commit 3ad900f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/spec/Listener/ForceHttpsSpec.php b/spec/Listener/ForceHttpsSpec.php
@@ -125,6 +125,7 @@
             allow($mvcEvent)->toReceive('getResponse')->andReturn($response);
             allow($response)->toReceive('setStatusCode')->with(302)->andReturn($response);
             allow($response)->toReceive('getHeaders', 'addHeaderLine')->with('Location', 'https://example.com/about');
+            allow($response)->toReceive('send');
 
             expect($mvcEvent)->toReceive('getResponse');
 

diff --git a/src/Listener/ForceHttps.php b/src/Listener/ForceHttps.php
@@ -64,5 +64,6 @@ public function forceHttpsScheme(MvcEvent $e)
         $response->setStatusCode(302);
         $response->getHeaders()
                  ->addHeaderLine('Location', $httpsRequestUri);
+        $response->send();
     }
 }