Make distinction between 0 or false for maxAge (#62)

Make distinction between 0 or null for maxAge

Author: barryvdh

src/Asm89/Stack/CorsService.php

@@ -116,7 +116,7 @@ private function buildPreflightCheckResponse(Request $request)
 
         $response->headers->set('Access-Control-Allow-Origin', $request->headers->get('Origin'));
 
-        if ($this->options['maxAge']) {
+        if ($this->options['maxAge'] !== null) {
             $response->headers->set('Access-Control-Max-Age', $this->options['maxAge']);
         }
 

test/Asm89/Stack/CorsTest.php

@@ -402,6 +402,33 @@ public function it_sets_max_age_when_set()
         $this->assertEquals(42, $response->headers->get('Access-Control-Max-Age'));
     }
 
+    /**
+     * @test
+     */
+    public function it_sets_max_age_when_zero()
+    {
+        $app     = $this->createStackedApp(array('maxAge' => 0));
+        $request = $this->createValidPreflightRequest();
+
+        $response = $app->handle($request);
+
+        $this->assertTrue($response->headers->has('Access-Control-Max-Age'));
+        $this->assertEquals(0, $response->headers->get('Access-Control-Max-Age'));
+    }
+
+    /**
+     * @test
+     */
+    public function it_doesnt_set_max_age_when_false()
+    {
+        $app     = $this->createStackedApp(array('maxAge' => null));
+        $request = $this->createValidPreflightRequest();
+
+        $response = $app->handle($request);
+
+        $this->assertFalse($response->headers->has('Access-Control-Max-Age'));
+    }
+
     /**
      * @test
      */