Skip to content

Commit bdd38db

Browse files
Merge pull request christoph-kluge#3 from WyriHaximus-labs/small-fixes
Small fixes/improvements
2 parents d847a41 + 6756561 commit bdd38db

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
},
2727
"require-dev": {
2828
"phpunit/phpunit": "^4.8.10||^5.0",
29-
"react/http": "dev-master",
29+
"react/http": "^0.8.1",
3030
"ringcentral/psr7": "^1.2"
3131
},
3232
"scripts": {

src/CorsMiddleware.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,10 @@
55
use Neomerx\Cors\Analyzer;
66
use Neomerx\Cors\Contracts\AnalysisResultInterface;
77
use Neomerx\Cors\Contracts\AnalyzerInterface;
8+
use Psr\Http\Message\ResponseInterface;
89
use Psr\Http\Message\ServerRequestInterface;
910
use React\Http\Response;
11+
use function React\Promise\resolve;
1012
use Sikei\React\Http\Middleware\CorsMiddlewareAnalysisStrategy as Strategy;
1113
use Sikei\React\Http\Middleware\CorsMiddlewareConfiguration as Config;
1214

@@ -47,7 +49,7 @@ public function __invoke(ServerRequestInterface $request, $next)
4749
return new Response(405, [], 'Method not supported');
4850
}
4951

50-
return $next($request)->then(function (Response $response) use ($cors) {
52+
return resolve($next($request))->then(function (ResponseInterface $response) use ($cors) {
5153
foreach ($cors->getResponseHeaders() as $header => $value) {
5254
$response = $response->withHeader($header, $value);
5355
}

tests/ResponseCompressionMiddlewareTest.php renamed to tests/CorsMiddlewareTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
use PHPUnit\Framework\TestCase;
77
use Psr\Http\Message\ServerRequestInterface;
88
use React\Http\Response;
9-
use React\Http\ServerRequest;
9+
use React\Http\Io\ServerRequest;
1010
use React\Promise\Promise;
1111
use React\Promise\PromiseInterface;
1212

0 commit comments

Comments
 (0)