@@ -222,7 +222,7 @@ |
||
| 222 | 222 | } |
| 223 | 223 | |
| 224 | 224 | if ($this->maxAge) { |
| 225 | - $response->headers->set('Access-Control-Max-Age', (string)$this->maxAge); |
|
| 225 | + $response->headers->set('Access-Control-Max-Age', (string) $this->maxAge); |
|
| 226 | 226 | } |
| 227 | 227 | |
| 228 | 228 | $allowMethods = $this->isAllMethodsAllowed() |
@@ -29,7 +29,7 @@ |
||
| 29 | 29 | protected function registerBindings() |
| 30 | 30 | { |
| 31 | 31 | // TODO: Change to bind the implementation to the interface instead. |
| 32 | - $this->app->bind(CorsService::class, function () { |
|
| 32 | + $this->app->bind(CorsService::class, function() { |
|
| 33 | 33 | return new CorsService(config(self::CONFIG_KEY)); |
| 34 | 34 | }); |
| 35 | 35 | |
@@ -33,8 +33,8 @@ discard block |
||
| 33 | 33 | */ |
| 34 | 34 | public function testAssertIsNotCorsRequest() |
| 35 | 35 | { |
| 36 | - $this->specify('verify middleware is not cors request', function () { |
|
| 37 | - verify($this->middleware->handle(new Request(), function () { |
|
| 36 | + $this->specify('verify middleware is not cors request', function() { |
|
| 37 | + verify($this->middleware->handle(new Request(), function() { |
|
| 38 | 38 | return true; |
| 39 | 39 | }))->equals(true); |
| 40 | 40 | }); |
@@ -45,10 +45,10 @@ discard block |
||
| 45 | 45 | */ |
| 46 | 46 | public function testAssertIsCorsRequest() |
| 47 | 47 | { |
| 48 | - $this->specify('verify middleware is cors request', function () { |
|
| 48 | + $this->specify('verify middleware is cors request', function() { |
|
| 49 | 49 | $req = new Request(); |
| 50 | 50 | $req->headers->set('Origin', 'http://example.com'); |
| 51 | - $res = $this->middleware->handle($req, function () { |
|
| 51 | + $res = $this->middleware->handle($req, function() { |
|
| 52 | 52 | return new JsonResponse(); |
| 53 | 53 | }); |
| 54 | 54 | |
@@ -60,10 +60,10 @@ discard block |
||
| 60 | 60 | 'allow_origins' => ['http://foo.com'], |
| 61 | 61 | ]); |
| 62 | 62 | $this->middleware = new CorsMiddleware($service); |
| 63 | - $this->specify('Closure is called even if origin is not allowed', function () { |
|
| 63 | + $this->specify('Closure is called even if origin is not allowed', function() { |
|
| 64 | 64 | $req = new Request(); |
| 65 | 65 | $req->headers->set('Origin', 'http://bar.com'); |
| 66 | - $res = $this->middleware->handle($req, function () { |
|
| 66 | + $res = $this->middleware->handle($req, function() { |
|
| 67 | 67 | $res = new JsonResponse(); |
| 68 | 68 | $res->headers->set('X-Closure-Called', 1); |
| 69 | 69 | return $res; |
@@ -78,13 +78,13 @@ discard block |
||
| 78 | 78 | */ |
| 79 | 79 | public function testAssertIsPreflightRequest() |
| 80 | 80 | { |
| 81 | - $this->specify('verify middleware is preflight request', function () { |
|
| 81 | + $this->specify('verify middleware is preflight request', function() { |
|
| 82 | 82 | $req = new Request(); |
| 83 | 83 | $req->setMethod('OPTIONS'); |
| 84 | 84 | $req->headers->set('Origin', 'http://example.com'); |
| 85 | 85 | $req->headers->set('Access-Control-Request-Method', 'GET'); |
| 86 | 86 | |
| 87 | - $res = $this->middleware->handle($req, function () { |
|
| 87 | + $res = $this->middleware->handle($req, function() { |
|
| 88 | 88 | return new JsonResponse(); |
| 89 | 89 | }); |
| 90 | 90 | |
@@ -28,7 +28,7 @@ |
||
| 28 | 28 | */ |
| 29 | 29 | public function testAssertCanBeRegistered() |
| 30 | 30 | { |
| 31 | - $this->specify('verify serviceProvider is registered', function () { |
|
| 31 | + $this->specify('verify serviceProvider is registered', function() { |
|
| 32 | 32 | $service = $this->app->make(CorsService::class); |
| 33 | 33 | verify($service)->isInstanceOf(CorsService::class); |
| 34 | 34 | }); |
@@ -38,7 +38,7 @@ discard block |
||
| 38 | 38 | |
| 39 | 39 | public function testServiceConfig() |
| 40 | 40 | { |
| 41 | - $this->specify('service config max_age is less than zero', function () { |
|
| 41 | + $this->specify('service config max_age is less than zero', function() { |
|
| 42 | 42 | new CorsService(['max_age' => -1]); |
| 43 | 43 | }, ['throws' => 'InvalidArgumentException']); |
| 44 | 44 | } |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | |
| 50 | 50 | $this->request = new Request; |
| 51 | 51 | |
| 52 | - $this->specify('403 response if origin is not allowed', function () { |
|
| 52 | + $this->specify('403 response if origin is not allowed', function() { |
|
| 53 | 53 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 54 | 54 | $this->request->headers->set('Access-Control-Request-Method', 'POST'); |
| 55 | 55 | $this->request->headers->set('Access-Control-Request-Headers', 'accept, authorization, content-type'); |
@@ -65,7 +65,7 @@ discard block |
||
| 65 | 65 | |
| 66 | 66 | $this->request = new Request; |
| 67 | 67 | |
| 68 | - $this->specify('405 response if method is not allowed', function () { |
|
| 68 | + $this->specify('405 response if method is not allowed', function() { |
|
| 69 | 69 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 70 | 70 | $this->request->headers->set('Access-Control-Request-Method', 'POST'); |
| 71 | 71 | $this->request->headers->set('Access-Control-Request-Headers', 'accept, authorization, content-type'); |
@@ -82,7 +82,7 @@ discard block |
||
| 82 | 82 | |
| 83 | 83 | $this->request = new Request; |
| 84 | 84 | |
| 85 | - $this->specify('403 response if header is not allowed', function () { |
|
| 85 | + $this->specify('403 response if header is not allowed', function() { |
|
| 86 | 86 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 87 | 87 | $this->request->headers->set('Access-Control-Request-Method', 'POST'); |
| 88 | 88 | $this->request->headers->set('Access-Control-Request-Headers', 'accept, authorization, content-type'); |
@@ -98,7 +98,7 @@ discard block |
||
| 98 | 98 | |
| 99 | 99 | $this->request = new Request; |
| 100 | 100 | |
| 101 | - $this->specify('200 response when origin, method and headers are allowed', function () { |
|
| 101 | + $this->specify('200 response when origin, method and headers are allowed', function() { |
|
| 102 | 102 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 103 | 103 | $this->request->headers->set('Access-Control-Request-Method', 'POST'); |
| 104 | 104 | $this->request->headers->set('Access-Control-Request-Headers', 'accept, authorization, content-type'); |
@@ -114,7 +114,7 @@ discard block |
||
| 114 | 114 | |
| 115 | 115 | $this->request = new Request; |
| 116 | 116 | |
| 117 | - $this->specify('403 response when origin is not set', function () { |
|
| 117 | + $this->specify('403 response when origin is not set', function() { |
|
| 118 | 118 | $response = $this->service->handlePreflightRequest($this->request); |
| 119 | 119 | |
| 120 | 120 | verify($response->getStatusCode())->equals(403); |
@@ -127,7 +127,7 @@ discard block |
||
| 127 | 127 | |
| 128 | 128 | $this->request = new Request; |
| 129 | 129 | |
| 130 | - $this->specify('403 response exception when header is not set', function () { |
|
| 130 | + $this->specify('403 response exception when header is not set', function() { |
|
| 131 | 131 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 132 | 132 | $this->request->headers->set('Access-Control-Request-Headers', 'accept, '); |
| 133 | 133 | |
@@ -144,7 +144,7 @@ discard block |
||
| 144 | 144 | |
| 145 | 145 | $this->request = new Request; |
| 146 | 146 | |
| 147 | - $this->specify('response headers are set', function () { |
|
| 147 | + $this->specify('response headers are set', function() { |
|
| 148 | 148 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 149 | 149 | $this->request->headers->set('Access-Control-Request-Method', 'POST'); |
| 150 | 150 | $this->request->headers->set('Access-Control-Request-Headers', 'accept, authorization, content-type'); |
@@ -166,7 +166,7 @@ discard block |
||
| 166 | 166 | |
| 167 | 167 | $this->request = new Request; |
| 168 | 168 | |
| 169 | - $this->specify('regression test for issue #31', function () { |
|
| 169 | + $this->specify('regression test for issue #31', function() { |
|
| 170 | 170 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 171 | 171 | $this->request->headers->set('Access-Control-Request-Method', 'POST'); |
| 172 | 172 | $this->request->headers->set('Access-Control-Request-Headers', 'accept,authorization, content-type'); |
@@ -189,7 +189,7 @@ discard block |
||
| 189 | 189 | |
| 190 | 190 | $this->request = new Request; |
| 191 | 191 | |
| 192 | - $this->specify('response credentials header is set', function () { |
|
| 192 | + $this->specify('response credentials header is set', function() { |
|
| 193 | 193 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 194 | 194 | $this->request->headers->set('Access-Control-Request-Method', 'POST'); |
| 195 | 195 | $this->request->headers->set('Access-Control-Request-Headers', 'accept, authorization, content-type'); |
@@ -208,7 +208,7 @@ discard block |
||
| 208 | 208 | |
| 209 | 209 | $this->request = new Request; |
| 210 | 210 | |
| 211 | - $this->specify('response max-age header is set', function () { |
|
| 211 | + $this->specify('response max-age header is set', function() { |
|
| 212 | 212 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 213 | 213 | $this->request->headers->set('Access-Control-Request-Method', 'POST'); |
| 214 | 214 | $this->request->headers->set('Access-Control-Request-Headers', 'accept, authorization, content-type'); |
@@ -225,7 +225,7 @@ discard block |
||
| 225 | 225 | |
| 226 | 226 | $this->response = new Response; |
| 227 | 227 | |
| 228 | - $this->closure = function () { |
|
| 228 | + $this->closure = function() { |
|
| 229 | 229 | return new Response; |
| 230 | 230 | }; |
| 231 | 231 | |
@@ -233,7 +233,7 @@ discard block |
||
| 233 | 233 | 'allow_origins' => ['*'], |
| 234 | 234 | ]); |
| 235 | 235 | |
| 236 | - $this->specify('response origin header is set', function () { |
|
| 236 | + $this->specify('response origin header is set', function() { |
|
| 237 | 237 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 238 | 238 | |
| 239 | 239 | $response = $this->service->handleRequest($this->request, $this->closure); |
@@ -245,7 +245,7 @@ discard block |
||
| 245 | 245 | 'allow_origins' => ['*'], |
| 246 | 246 | ]); |
| 247 | 247 | |
| 248 | - $this->specify('response vary header is set', function () { |
|
| 248 | + $this->specify('response vary header is set', function() { |
|
| 249 | 249 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 250 | 250 | $this->request->headers->set('Vary', 'Accept-Encoding'); |
| 251 | 251 | |
@@ -261,7 +261,7 @@ discard block |
||
| 261 | 261 | 'allow_credentials' => true, |
| 262 | 262 | ]); |
| 263 | 263 | |
| 264 | - $this->specify('response credentials header is set', function () { |
|
| 264 | + $this->specify('response credentials header is set', function() { |
|
| 265 | 265 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 266 | 266 | |
| 267 | 267 | $response = $this->service->handleRequest($this->request, $this->closure); |
@@ -276,7 +276,7 @@ discard block |
||
| 276 | 276 | 'expose_headers' => ['Accept', 'Authorization', 'Content-Type'], |
| 277 | 277 | ]); |
| 278 | 278 | |
| 279 | - $this->specify('response expose headers header is set', function () { |
|
| 279 | + $this->specify('response expose headers header is set', function() { |
|
| 280 | 280 | $this->request->headers->set('Origin', 'http://foo.com'); |
| 281 | 281 | |
| 282 | 282 | $response = $this->service->handleRequest($this->request, $this->closure); |
@@ -288,7 +288,7 @@ discard block |
||
| 288 | 288 | 'allow_origins' => ['http://foo.com'], |
| 289 | 289 | ]); |
| 290 | 290 | |
| 291 | - $this->specify('response origin header is not set when origin is not allowed', function () { |
|
| 291 | + $this->specify('response origin header is not set when origin is not allowed', function() { |
|
| 292 | 292 | $this->request->headers->set('Origin', 'http://bar.com'); |
| 293 | 293 | |
| 294 | 294 | $response = $this->service->handleRequest($this->request, $this->closure); |
@@ -304,7 +304,7 @@ discard block |
||
| 304 | 304 | |
| 305 | 305 | $this->request = new Request; |
| 306 | 306 | |
| 307 | - $this->specify('cors request is recognized', function () { |
|
| 307 | + $this->specify('cors request is recognized', function() { |
|
| 308 | 308 | verify($this->service->isCorsRequest($this->request))->false(); |
| 309 | 309 | |
| 310 | 310 | $this->request->headers->set('Origin', 'http://foo.com'); |
@@ -319,7 +319,7 @@ discard block |
||
| 319 | 319 | |
| 320 | 320 | $this->request = new Request; |
| 321 | 321 | |
| 322 | - $this->specify('preflight request is recognized', function () { |
|
| 322 | + $this->specify('preflight request is recognized', function() { |
|
| 323 | 323 | verify($this->service->isPreflightRequest($this->request))->false(); |
| 324 | 324 | |
| 325 | 325 | $this->request->setMethod('OPTIONS'); |