Merge pull request #156 from splitio/develop

Develop

Author: mmelograno

.travis.yml

@@ -19,6 +19,9 @@ matrix:
 services:
   - redis-server
 
+before-install:
+  - echo "extension = zip.so" >> ~/.phpenv/versions/$(phpenv version-name)/etc/php.ini
+
 install:
   - composer install --prefer-source --no-interaction
   - composer update

CHANGES.txt

@@ -1,3 +1,9 @@
+6.2.5 (Apr 27, 2021)
+ - Added delimiter in SharedMemory Cache.
+
+6.2.4 (Mar 20, 2020)
+ - Added missing methods in ClientInterface.
+
 6.2.3 (Nov 1, 2019)
  - Added flag `IPAddressesEnabled` into options to enable/disable sending MachineName and MachineIP when data is posted in headers.
 

Detailed-README.md

@@ -391,7 +391,7 @@ $splitClient = $splitFactory->client();
 Within tests folder you can find different test suites in order to run the Split SDK tests. The most important test suite is: **integration** that wrap the others test suites.
 
 ### Integration test suite
-Before to run this test suite, please be sure to have a Redis instance runing:
+Before to run this test suite, please be sure to have a Redis instance running:
 - In order to have a local Redis instance you can install [Docker Container Tool](https://www.docker.com) and pull the oficial Redis container running the command ```docker pull redis```.
 
 And set the correct values on the **phpunit.xml** that you should have copied from **phpunit.xml.dist** file. 

README.md

@@ -49,7 +49,7 @@ Split has built and maintains SDKs for:
 * Java [Github](https://github.com/splitio/java-client) [Docs](https://help.split.io/hc/en-us/articles/360020405151-Java-SDK)
 * Javascript [Github](https://github.com/splitio/javascript-client) [Docs](https://help.split.io/hc/en-us/articles/360020448791-JavaScript-SDK)
 * Node [Github](https://github.com/splitio/javascript-client) [Docs](https://help.split.io/hc/en-us/articles/360020564931-Node-js-SDK)
-* .NET [Github](https://github.com/splitio/.net-core-client) [Docs](https://help.split.io/hc/en-us/articles/360020240172--NET-SDK)
+* .NET [Github](https://github.com/splitio/dotnet-client) [Docs](https://help.split.io/hc/en-us/articles/360020240172--NET-SDK)
 * Ruby [Github](https://github.com/splitio/ruby-client) [Docs](https://help.split.io/hc/en-us/articles/360020673251-Ruby-SDK)
 * PHP [Github](https://github.com/splitio/php-client) [Docs](https://help.split.io/hc/en-us/articles/360020350372-PHP-SDK)
 * Python [Github](https://github.com/splitio/python-client) [Docs](https://help.split.io/hc/en-us/articles/360020359652-Python-SDK)

composer.json

@@ -33,7 +33,7 @@
     "phpunit/phpunit": "~4.0",
     "phpdocumentor/phpdocumentor": "2.*",
     "squizlabs/php_codesniffer": "2.*",
-    "rogervila/php-sonarqube-scanner": "0.4.0"
+    "rogervila/php-sonarqube-scanner": "1.1.0"
   },
 
   "autoload": {

src/SplitIO/Component/Common/ErrorHandler.php

@@ -74,7 +74,7 @@ public static function stop($throw = false)
     }
 
     /**
-     * Method to add the catched error
+     * Method to add the caught error
      * @param $errno
      * @param string $errstr
      * @param string $errfile

src/SplitIO/Component/Memory/SharedMemory.php

@@ -37,7 +37,7 @@ public static function write($key, $value, $expiration = 60, $mode = 0644, $size
 
         $expiration = time() + $expiration;
 
-        $data = json_encode(array('expiration' => $expiration, 'value' => serialize($value)));
+        $data = json_encode(array('expiration' => $expiration, 'value' => serialize($value))) . "\0";
 
         @$shm_id = shmop_open($key, "c", $mode, $size);
 
@@ -87,7 +87,11 @@ public static function read($key, $mode = 0644, $size = 100)
             throw new ReadSharedMemoryException("The shared memory block could not be read");
         }
 
-        $data = json_decode($cached_string, true);
+        $null_pos = strpos($cached_string, "\0");
+        $data = json_decode(
+            substr($cached_string, 0, $null_pos),
+            true
+        );
 
         if ((isset($data['expiration']) && time() > $data['expiration']) || !isset($data['expiration'])) {
             shmop_delete($shm_id);

src/SplitIO/Sdk/Client.php

@@ -254,7 +254,7 @@ private function registerData($impressions, $attributes, $metricName, $latency =
             }
         } catch (\Exception $e) {
             SplitApp::logger()->critical(
-                ': An exception occured when trying to store impressions.'
+                ': An exception occurred when trying to store impressions.'
             );
         }
     }

src/SplitIO/Sdk/Evaluator.php

@@ -117,7 +117,7 @@ private function evalTreatment($key, $bucketingKey, $split, array $attributes =
             }
             SplitApp::logger()->info("*Treatment for $key in {$split->getName()} is: ".$result['treatment']);
         } catch (\Exception $e) {
-            SplitApp::logger()->critical('An exception occured when evaluating feature: '. $split->getName());
+            SplitApp::logger()->critical('An exception occurred when evaluating feature: '. $split->getName());
             SplitApp::logger()->critical($e->getMessage());
             SplitApp::logger()->critical($e->getTraceAsString());
             $result['impression']['label'] = ImpressionLabel::EXCEPTION;

src/SplitIO/Version.php

@@ -3,5 +3,5 @@
 
 class Version
 {
-    const CURRENT = '6.2.4';
+    const CURRENT = '6.2.5';
 }