v1.5.1

Author: cheprasov

CHANGELOG.md

@@ -1,5 +1,8 @@
 ## CHANGELOG
 
+### v1.5.1 (2016-08-17)
+- Fixed critical bug: https://github.com/cheprasov/php-redis-client/pull/45 Thanks to @BrianFranklin for help.
+
 ### v1.5.0 (2016-08-15)
 - Added command **TOUCH** for Redis >= 3.2.1
 

README.md

@@ -1,7 +1,7 @@
 [![MIT license](http://img.shields.io/badge/license-MIT-brightgreen.svg)](http://opensource.org/licenses/MIT)
 [![Latest Stable Version](https://poser.pugx.org/cheprasov/php-redis-client/v/stable)](https://packagist.org/packages/cheprasov/php-redis-client)
 [![Total Downloads](https://poser.pugx.org/cheprasov/php-redis-client/downloads)](https://packagist.org/packages/cheprasov/php-redis-client)
-# RedisClient v1.5.0 for PHP >= 5.5
+# RedisClient v1.5.1 for PHP >= 5.5
 
 ## About
 RedisClient is a fast, fully-functional and user-friendly client for Redis, optimized for performance. RedisClient supports the latest versions of Redis starting from __2.6__ to __3.2__
@@ -14,7 +14,7 @@ RedisClient is a fast, fully-functional and user-friendly client for Redis, opti
 - Support __RAW__ commands as strings `"SET foo bar"` or as arrays `['SET', 'foo', 'bar']`.
 - Connections to Redis are established lazily by the client upon the first command.
 - Easy to use with IDE, client has PHPDocs for all supported versions.
-- By default, the client works with the latest stable version of Redis (3.2.0).
+- By default, the client works with the latest stable version of Redis (3.2.3).
 - About **6.5-8.5% faster** than predis (based on this test: https://github.com/cheprasov/php-redis-client-vs-predis-test)
 
 ## Usage

composer.json

@@ -1,6 +1,6 @@
 {
   "name": "cheprasov/php-redis-client",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "Php client for Redis. It is a fast, fully-functional and user-friendly client for Redis, optimized for performance. RedisClient supports the latest versions of Redis starting from 2.6 to 3.2",
   "homepage": "http://github.com/cheprasov/php-redis-client",
   "minimum-stability": "stable",

src/RedisClient/Client/AbstractRedisClient.php

@@ -21,7 +21,7 @@
 
 abstract class AbstractRedisClient {
 
-    const VERSION = '1.5.0';
+    const VERSION = '1.5.1';
 
     const CONFIG_SERVER = 'server';
     const CONFIG_TIMEOUT = 'timeout';

src/RedisClient/Connection/StreamConnection.php

@@ -87,12 +87,14 @@ public function readLine() {
      * @return string
      */
     public function read($length) {
-        $read = 0;
+        $resource = $this->getResource();
+        $left = $length;
         $data = '';
-        while ($read != $length) {
-            $data .= fread($this->getResource(), ($length - $read));
-            $read = strlen($data);
-        }
+        do {
+            $data .= fread($resource, min($left, 8192));
+            $left = $length - strlen($data);
+        } while ($left > 0);
+
         return $data;
     }
 

tests/Integration/Version2x6/CommonTest.php

@@ -84,4 +84,15 @@ public function test_executeRaw() {
         $this->assertSame("String\r\nwith\r\nnewlines", $Redis->executeRaw(['GET', '']));
     }
 
+    public function test_bigData() {
+        $Redis = static::$Redis;
+
+        $string = str_repeat(microtime(true), 1024 * 1024);
+        $md5 = md5($string);
+
+        $this->assertSame(true, $Redis->set('foo', $string));
+        $this->assertSame($string, $Redis->get('foo'));
+        $this->assertSame($md5, md5($Redis->get('foo')));
+    }
+
 }