fix: debug events, multi instance, deprecation warnings

Author: Kaustubh22327

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.11.0] - 2026-08-06
+
+### Fixed
+
+- Fixed several bugs: wrong user ID in debug events, broken tracking URLs with special characters, mixed-up credentials when running multiple SDK instances, failed Gateway requests for some accounts, and PHP 8.4/8.5 deprecation warnings.
+
 ## [2.10.0] - 2026-07-04
 
 ### Added

composer.json

@@ -1,6 +1,6 @@
 {
     "name": "wingify/wingify-fme-php-sdk",
-    "version": "2.10.0",
+    "version": "2.11.0",
     "keywords": [
         "wingify",
         "fme",

src/Api/GetFlag.php

@@ -623,8 +623,8 @@ public function get(
             // Update debug event props with decision keys
             $this->updateDebugEventPropsWithDecisionKeys($debugEventProps, $decision);
 
-            // Send debug event
-            DebuggerServiceUtil::sendDebugEvent($debugEventProps);
+            // Send debug event scoped to this SDK instance (multi-instance safe).
+            DebuggerServiceUtil::sendDebugEvent($debugEventProps, $serviceContainer);
        }
 
         // Send data for Impact Campaign, if defined

src/Api/SetAttribute.php

@@ -43,7 +43,7 @@ class SetAttribute implements ISetAttribute
      * @param array $attributes Key-value map of attributes.
      * @param ContextModel $context Context containing user information.
      */
-    public function setAttribute(SettingsModel $settings, array $attributes, ContextModel $context, bool $isDebuggerUsed = false, ServiceContainer $serviceContainer = null)
+    public function setAttribute(SettingsModel $settings, array $attributes, ContextModel $context, bool $isDebuggerUsed = false, $serviceContainer = null)
     {
         if (!$isDebuggerUsed) {
             $this->createImpressionForAttributes($settings, $attributes, $context, $serviceContainer);
@@ -57,7 +57,8 @@ public function setAttribute(SettingsModel $settings, array $attributes, Context
      * @param ContextModel $context Context containing user information.
      * @param ServiceContainer $serviceContainer The service container (optional).
      */
-    private function createImpressionForAttributes(SettingsModel $settings, array $attributes, ContextModel $context, ServiceContainer $serviceContainer = null)
+    /** @param ServiceContainer|null $serviceContainer */
+    private function createImpressionForAttributes(SettingsModel $settings, array $attributes, ContextModel $context, $serviceContainer = null)
     {
         $networkUtil = new NetworkUtil($serviceContainer);
 

src/Api/TrackEvent.php

@@ -54,7 +54,7 @@ class TrackEvent implements ITrack
      * @param HooksService $hooksService Manager for handling hooks and callbacks.
      * @return array Returns an array indicating the success or failure of the event tracking.
      */
-    public function track(SettingsModel $settings, string $eventName, ContextModel $context, array $eventProperties, HooksService $hooksService, bool $isDebuggerUsed = false, ServiceContainer $serviceContainer = null): array
+    public function track(SettingsModel $settings, string $eventName, ContextModel $context, array $eventProperties, HooksService $hooksService, bool $isDebuggerUsed = false, $serviceContainer = null): array
     {
         if (FunctionUtil::doesEventBelongToAnyFeature($eventName, $settings) || FunctionUtil::doesEventBelongToAnyHoldout($eventName, $settings)) {
             // Create an impression for the track event
@@ -90,7 +90,8 @@ public function track(SettingsModel $settings, string $eventName, ContextModel $
      * @param array $eventProperties Properties associated with the event.
      * @param ServiceContainer $serviceContainer The service container (optional).
      */
-    private function createImpressionForTrack(SettingsModel $settings, string $eventName, ContextModel $context, array $eventProperties, ServiceContainer $serviceContainer = null)
+    /** @param ServiceContainer|null $serviceContainer */
+    private function createImpressionForTrack(SettingsModel $settings, string $eventName, ContextModel $context, array $eventProperties, $serviceContainer = null)
     {
         $networkUtil = new NetworkUtil($serviceContainer);
 

src/Constants/Constants.php

@@ -49,7 +49,7 @@ class Constants {
     const DEFAULT_EVENTS_PER_REQUEST = 100;
     const SDK_NAME = 'wingify-fme-php-sdk';
 
-    const SDK_VERSION = '2.10.0';
+    const SDK_VERSION = '2.11.0';
     const AP = 'server';
 
     const SETTINGS = 'settings';

src/Decorators/StorageDecorator.php

@@ -30,13 +30,16 @@
 
 interface IStorageDecorator
 {
-    public function getFeatureFromStorage($featureKey, $context, $storageService, ServiceContainer $serviceContainer = null);
-    public function setDataInStorage($data, $storageService, ServiceContainer $serviceContainer = null);
+    /** @param ServiceContainer|null $serviceContainer */
+    public function getFeatureFromStorage($featureKey, $context, $storageService, $serviceContainer = null);
+    /** @param ServiceContainer|null $serviceContainer */
+    public function setDataInStorage($data, $storageService, $serviceContainer = null);
 }
 
 class StorageDecorator implements IStorageDecorator
 {
-    public function getFeatureFromStorage($featureKey, $context, $storageService, ServiceContainer $serviceContainer = null)
+    /** @param ServiceContainer|null $serviceContainer */
+    public function getFeatureFromStorage($featureKey, $context, $storageService, $serviceContainer = null)
     {
         $campaignMap = $storageService->getDataInStorage($featureKey, $context, $serviceContainer);
 
@@ -58,7 +61,8 @@ public function getFeatureFromStorage($featureKey, $context, $storageService, Se
         }
     }
 
-    public function setDataInStorage($data, $storageService, ServiceContainer $serviceContainer = null)
+    /** @param ServiceContainer|null $serviceContainer */
+    public function setDataInStorage($data, $storageService, $serviceContainer = null)
     {
         $featureKey = $data['featureKey'] ?? null;
         $featureId = $data['featureId'] ?? null;

src/Packages/NetworkLayer/Client/NetworkClient.php

@@ -116,7 +116,10 @@ private function makeCurlRequest($request, $url, $method, $headers, $body = null
             $response = curl_exec($ch);
             $httpCode = curl_getinfo($ch, CURLINFO_HTTP_CODE);
             $error = curl_error($ch);
-            curl_close($ch);
+            // For PHP < 8.0, $ch is a resource and requires curl_close. For PHP >= 8.0, it's a CurlHandle object and closes automatically.
+            if (is_resource($ch)) {
+                curl_close($ch);
+            }
 
             $responseModel = new ResponseModel($response, $httpCode, $error);
 

src/Packages/NetworkLayer/Models/RequestModel.php

@@ -146,7 +146,11 @@ public function setPath($path) {
     public function getOptions() {
         $queryParams = '';
         foreach ($this->query as $key => $value) {
-            $queryParams .= "{$key}={$value}&";
+            if ($value === null) {
+                continue;
+            }
+            // Encode query values (e.g. visitor_ua) so special characters do not break trackEvent/setAttribute URLs.
+            $queryParams .= rawurlencode((string) $key) . '=' . rawurlencode((string) $value) . '&';
         }
         $queryParams = rtrim($queryParams, '&');
 

src/Packages/SegmentationEvaluator/Core/SegmentationManager.php

@@ -52,11 +52,12 @@ public function setContextualData(ServiceContainer $serviceContainer, $feature,
         $settings = $serviceContainer->getSettings();
         $loggerService = $serviceContainer->getLoggerService();
 
-        $this->attachEvaluator(); // Ensure a fresh evaluator instance
+        // Pass serviceContainer into SegmentEvaluator so SegmentOperandEvaluator is also wired.
+        // Previously only $this->evaluator->serviceContainer was set, leaving
+        // segmentOperandEvaluator->serviceContainer null and causing
+        // "Call to a member function getLogManager() on null" during holdout/IP/UA segmentation.
+        $this->attachEvaluator(new SegmentEvaluator($serviceContainer, $context, $feature));
         $this->evaluator->settings = $settings; // Set settings in evaluator
-        $this->evaluator->serviceContainer = $serviceContainer; // Set serviceContainer in evaluator
-        $this->evaluator->context = $context; // Set context in evaluator
-        $this->evaluator->feature = $feature; // Set feature in evaluator
 
         // if both user agent and ip is null then we should not get data from gateway service
         if ($context->getUserAgent() === null && $context->getIpAddress() === null) {