Make sure independent from ANY PSR7 implementation

[Nyholm]

PSR-17 (HTTP factories) has just been accepted. We should support ANY PSR-7 implementation.

Since we are working with ServerRequests it is a little more tricky. PSR-17 does not really help much here. I've created a "helper package" so one easily can create server requests. See https://github.com/Nyholm/psr7-server

---

With this PR:

Install nyholm/psr7-server and any PSR7 and PSR17 implementation.

composer require nyholm/psr7-server nyholm/psr7

use PHPFastCGI\FastCGIDaemon\ApplicationFactory;
use PHPFastCGI\FastCGIDaemon\Http\Request;
use Nyholm\Psr7Server\ServerRequestCreator;

// Could be any PSR7 implementation
use Nyholm\Psr7\Factory\Psr17Factory;
use Nyholm\Psr7\Response;

$psr17Factory = new Psr17Factory();

Request::setServerRequestCreator(new ServerRequestCreator(
    $psr17Factory,   // ServerRequestFactoryInteface 
    $psr17Factory,   // UriFactoryInterface
    $psr17Factory,   // UploadedFileFactoryInterface
    $psr17Factory    // StreamFactoryInterface
));

$kernel = function (RequestInterface $request) {
    $psr7Request = $request->getServerRequest();

    return new Response('<h1>Url, '.$psr7Request->getUri().'</h1>');
};

$application = (new ApplicationFactory)->createApplication($kernel);

$application->run();

[Nyholm]

Will be 0.1.2 shortly

[Nyholm]

I will release 1.0.0 when the psr for HTTP factories is released as a package (within few days)

[AndrewCarterUK]

Looks good, but won't that clash with this line which uses Zend\Diactoros\createUploadedFile?

I'm also wondering if there's a sensible way for doing defaults, or some sort of helper function that removes the boiler plate.

[Nyholm]

Yes, that line is not needed anymore after this PR is merged.

I'm also wondering if there's a sensible way for doing defaults, or some sort of helper function that removes the boiler plate.

Im not sure. The ServerRequestCreator needs you to give it 4 factory implementations... And we cannot do a "default" value without being dependent on a PSR-17 implementation.

[coveralls]

Coverage decreased (-0.4%) to 99.018% when pulling ae21ab3 on Nyholm:psr7-independent into 01a8f0b on PHPFastCGI:master.

[Nyholm]

PR is rebased and dependecies are updated

[AndrewCarterUK]

Looks good - I'm happy to merge.

The README.md file will need updating too, or do that before tagging?

[Nyholm]

I vote for a merge now. I’ll update the docs before tagging.