Send correct status code regardless of location header

PHP thinks it's clever, but it isn't. Fixes #1730.
slimphp · Nov 3, 2017 · 0dab1d1 · 0dab1d1
1 parent d1c5b02
commit 0dab1d1
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/Slim/App.php b/Slim/App.php
@@ -664,20 +664,23 @@ public function respond(ResponseInterface $response)
     {
         // Send response
         if (!headers_sent()) {
+            // Headers
+            foreach ($response->getHeaders() as $name => $values) {
+                foreach ($values as $value) {
+                    header(sprintf('%s: %s', $name, $value), false);
+                }
+            }
+
+            // Set the status _after_ the headers, because of PHP's "helpful" behavior with location headers.
+            // See https://github.com/slimphp/Slim/issues/1730
+
             // Status
             header(sprintf(
                 'HTTP/%s %s %s',
                 $response->getProtocolVersion(),
                 $response->getStatusCode(),
                 $response->getReasonPhrase()
             ));
-
-            // Headers
-            foreach ($response->getHeaders() as $name => $values) {
-                foreach ($values as $value) {
-                    header(sprintf('%s: %s', $name, $value), false);
-                }
-            }
         }
 
         // Body