Make sure we do not rewind a non-seekable stream

Author: Nyholm

spec/StreamFactory/DiactorosStreamFactorySpec.php

@@ -19,4 +19,12 @@ function it_creates_a_stream_from_stream()
         $this->createStream(new Stream('php://memory'))
             ->shouldHaveType('Psr\Http\Message\StreamInterface');
     }
+
+    function it_creates_a_stream_from_non_seekable_resource()
+    {
+        $url = 'https://raw.githubusercontent.com/php-http/multipart-stream-builder/master/tests/Resources/httplug.png';
+        $resource = fopen($url, 'r');
+        $this->createStream($resource)
+            ->shouldHaveType('Psr\Http\Message\StreamInterface');
+    }
 }

spec/StreamFactory/GuzzleStreamFactorySpec.php

@@ -19,4 +19,12 @@ public function it_creates_a_stream_from_stream()
         $this->createStream(new Stream(fopen('php://memory', 'rw')))
             ->shouldHaveType('Psr\Http\Message\StreamInterface');
     }
+
+    function it_creates_a_stream_from_non_seekable_resource()
+    {
+        $url = 'https://raw.githubusercontent.com/php-http/multipart-stream-builder/master/tests/Resources/httplug.png';
+        $resource = fopen($url, 'r');
+        $this->createStream($resource)
+            ->shouldHaveType('Psr\Http\Message\StreamInterface');
+    }
 }

spec/StreamFactory/SlimStreamFactorySpec.php

@@ -23,4 +23,12 @@ function it_creates_a_stream_from_stream()
         $this->createStream(new Stream($resource))
             ->shouldHaveType('Psr\Http\Message\StreamInterface');
     }
+
+    function it_creates_a_stream_from_non_seekable_resource()
+    {
+        $url = 'https://raw.githubusercontent.com/php-http/multipart-stream-builder/master/tests/Resources/httplug.png';
+        $resource = fopen($url, 'r');
+        $this->createStream($resource)
+            ->shouldHaveType('Psr\Http\Message\StreamInterface');
+    }
 }

src/StreamFactory/DiactorosStreamFactory.php

@@ -34,7 +34,9 @@ public function createStream($body = null)
             }
         }
 
-        $body->rewind();
+        if ($body->isSeekable()) {
+            $body->rewind();
+        }
 
         return $body;
     }

src/StreamFactory/SlimStreamFactory.php

@@ -35,7 +35,9 @@ public function createStream($body = null)
             $stream->write((string) $body);
         }
 
-        $stream->rewind();
+        if ($stream->isSeekable()) {
+            $stream->rewind();
+        }
 
         return $stream;
     }