Feature/http2 support

docs/book/client/intro.md

@@ -77,7 +77,7 @@ Parameter         | Description
 `strictredirects` | Whether to strictly follow the RFC when redirecting (see this section)                                                                                                               | boolean         | FALSE
 `useragent`       | User agent identifier string (sent in request headers)                                                                                                                               | string          | `Zend\Http\Client`
 `timeout`         | Connection timeout (seconds)                                                                                                                                                         | integer         | 10
-`httpversion`     | HTTP protocol version (usually '1.1' or '1.0')                                                                                                                                       | string          | 1.1
+`httpversion`     | HTTP protocol version (usually '1.1', '1.0' or '2.0')                                                                                                                                | string          | 1.1
 `adapter`         | Connection adapter class to use (see this section)                                                                                                                                   | mixed           | `Zend\Http\Client\Adapter\Socket`
 `keepalive`       | Whether to enable keep-alive connections with the server. Useful and might improve performance if several consecutive requests to the same server are performed.                     | boolean         | FALSE
 `storeresponse`   | Whether to store last response for later retrieval with getLastResponse(). If set to FALSE, getLastResponse() will return NULL.                                                      | boolean         | TRUE

docs/book/request.md

@@ -75,7 +75,7 @@ Method signature                                                            | De
 `setUri(string|Uri $uri) : self`                                            | Set the URI/URL for this request; this can be a string or an instance of `Zend\Uri\Http`.
 `getUri() : Uri`                                                            | Return the URI for this request object.
 `getUriString() : string`                                                   | Return the URI for this request object as a string.
-`setVersion(string $version) : self`                                        | Set the HTTP version for this object, one of 1.0 or 1.1 (`Request::VERSION_10`, `Request::VERSION_11`).
+`setVersion(string $version) : self`                                        | Set the HTTP version for this object, one of 1.0, 1.1 or 2 (`Request::VERSION_10`, `Request::VERSION_11`, `Request::VERSION_2`).
 `getVersion() : string`                                                     | Return the HTTP version for this request.
 `setQuery(Parameters $query) : self`                                        | Provide an alternate Parameter Container implementation for query parameters in this object. (This is NOT the primary API for value setting; for that, see `getQuery()`).
 `getQuery(string|null $name, mixed|null $default) : null|string|Parameters` | Return the parameter container responsible for query parameters or a single query parameter based on `$name`.

docs/book/response.md

@@ -77,8 +77,8 @@ Method signature                                                       | Descrip
 `renderStatusLine() : string`                                          | Render the status line header
 `setHeaders(Headers $headers) : self`                                  | Provide an alternate Parameter Container implementation for headers in this object. (This is NOT the primary API for value setting; for that, see `getHeaders()`.)
 `getHeaders() : Headers`                                               | Return the container responsible for storing HTTP headers. This container exposes the primary API for manipulating headers set in the HTTP response. See the section on [Headers](headers.md) for more information.
-`setVersion(string $version) : self`                                   | Set the HTTP version for this object, one of 1.0 or 1.1 (`Request::VERSION_10`, `Request::VERSION_11`).
-`getVersion() : string`                                                | Return the HTTP version for this request.
+`setVersion(string $version) : self`                                   | Set the HTTP version for this object, one of 1.0, 1.1 or 2 (`Response::VERSION_10`, `Response::VERSION_11`, `Response::VERSION_2`).
+`getVersion() : string`                                                | Return the HTTP version for this response.
 `setStatusCode(int $code) : self`                                      | Set HTTP status code.
 `getStatusCode() : int`                                                | Retrieve HTTP status code.
 `setReasonPhrase(string $reasonPhrase) : self`                         | Set custom HTTP status message.

src/AbstractMessage.php

@@ -21,6 +21,7 @@ abstract class AbstractMessage extends Message
      */
     const VERSION_10 = '1.0';
     const VERSION_11 = '1.1';
+    const VERSION_2  = '2';
     /**#@-*/
 
     /**
@@ -34,16 +35,16 @@ abstract class AbstractMessage extends Message
     protected $headers;
 
     /**
-     * Set the HTTP version for this object, one of 1.0 or 1.1
-     * (AbstractMessage::VERSION_10, AbstractMessage::VERSION_11)
+     * Set the HTTP version for this object, one of 1.0, 1.1 or 2
+     * (AbstractMessage::VERSION_10, AbstractMessage::VERSION_11, AbstractMessage::VERSION_2)
      *
-     * @param  string $version (Must be 1.0 or 1.1)
+     * @param  string $version (Must be 1.0, 1.1 or 2)
      * @return AbstractMessage
      * @throws Exception\InvalidArgumentException
      */
     public function setVersion($version)
     {
-        if ($version != self::VERSION_10 && $version != self::VERSION_11) {
+        if ($version != self::VERSION_10 && $version != self::VERSION_11 && $version != self::VERSION_2) {
             throw new Exception\InvalidArgumentException(
                 'Not valid or not supported HTTP version: ' . $version
             );

src/Response.php

@@ -249,7 +249,7 @@ public static function fromString($string)
      */
     protected function parseStatusLine($line)
     {
-        $regex   = '/^HTTP\/(?P<version>1\.[01]) (?P<status>\d{3})(?:[ ]+(?P<reason>.*))?$/';
+        $regex   = '/^HTTP\/(?P<version>1\.[01]|2) (?P<status>\d{3})(?:[ ]+(?P<reason>.*))?$/';
         $matches = [];
         if (! preg_match($regex, $line, $matches)) {
             throw new Exception\InvalidArgumentException(

test/ResponseTest.php

@@ -22,17 +22,32 @@ public function testResponseFactoryFromStringCreatesValidResponse()
         $response = Response::fromString($string);
         $this->assertEquals(200, $response->getStatusCode());
         $this->assertEquals('Foo Bar', $response->getContent());
+
+        $string = 'HTTP/2 200 OK' . "\r\n\r\n" . 'Foo Bar';
+        $response = Response::fromString($string);
+        $this->assertEquals('2', $response->getVersion());
     }
 
     public function testResponseCanRenderStatusLine()
     {
         $response = new Response();
-        $response->setVersion(1.1);
+        $response->setVersion('1.1');
         $response->setStatusCode(Response::STATUS_CODE_404);
         $this->assertEquals('HTTP/1.1 404 Not Found', $response->renderStatusLine());
 
         $response->setReasonPhrase('Foo Bar');
         $this->assertEquals('HTTP/1.1 404 Foo Bar', $response->renderStatusLine());
+
+        $response->setVersion('2');
+        $this->assertEquals('HTTP/2 404 Foo Bar', $response->renderStatusLine());
+    }
+
+    public function testInvalidHTTPVersion()
+    {
+        $this->expectException(InvalidArgumentException::class);
+        $this->expectExceptionMessage('A valid response status line was not found in the provided string');
+        $string = 'HTTP/2.0 200 OK' . "\r\n\r\n" . 'Foo Bar';
+        $response = \Zend\Http\Response::fromString($string);
     }
 
     public function testResponseUsesHeadersContainerByDefault()