fix: support for timeout in settings, ansi color disable

sakshamg1304 · sakshamg1304 · commit 23ceddeba4ce · 2026-01-09T17:45:41.000+05:30
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,38 @@ 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).
 
+## [1.17.0] - 2026-01-09
+
+### Added
+
+- Added support for configurable ANSI color output in logging. ANSI colors in logging are now only applied when `isAnsiColorEnabled` is set to `true` in the logger configuration.
+
+Example:
+
+```php
+$vwoClient = VWO::init([
+    'accountId' => '123456',
+    'sdkKey' => '32-alpha-numeric-sdk-key',
+    'logger' => [
+        'level' => 'DEBUG',
+        'isAnsiColorEnabled' => true, // Enable colored log levels in terminal
+    ],
+]);
+```
+
+- Added support for configurable settings expiry and network timeout through `settingsConfig` option. The SDK now allows customization of settings cache expiration and network timeout for settings fetch requests.
+
+Example:
+
+```php
+$vwoClient = VWO::init([
+    'accountId' => '123456',
+    'sdkKey' => '32-alpha-numeric-sdk-key',
+    'settingsConfig' => [
+        'timeout' => 50000,       // Network timeout for settings fetch in milliseconds (default: 50000)
+    ],
+]);
+```
 
 ## [1.16.0] - 2025-12-12
 
diff --git a/README.md b/README.md
@@ -315,6 +315,7 @@ To gain more control over VWO's logging behaviour, you can use the `logger` para
 | ------------- | -------------------------------------- | ------------ | -------- | --------------------- |
 | `level`       | Log level to control verbosity of logs | Yes          | string   | `'DEBUG'`             |
 | `prefix`      | Custom prefix for log messages         | No           | string   | `'CUSTOM LOG PREFIX'` |
+| `isAnsiColorEnabled` | Enable ANSI color codes for log levels in terminal output | No | boolean | `true` |
 | `transport`   | Custom logger implementation           | No           | array    | See example below     |
 
 #### Example 1: Set log level to control verbosity of logs
@@ -342,7 +343,22 @@ $vwoClient2 = VWO::init([
 ]);
 ```
 
-#### Example 3: Implement custom transport to handle logs your way
+#### Example 3: Enable ANSI color codes for colored log levels in terminal
+
+```php
+$vwoClient3 = VWO::init([
+  'sdkKey' => '32-alpha-numeric-sdk-key',
+  'accountId' => '123456',
+  'logger' => [
+    'level' => 'DEBUG',
+    'isAnsiColorEnabled' => true, // Enable colored log levels (TRACE, DEBUG, INFO, WARN, ERROR)
+  ],
+]);
+```
+
+Note: ANSI color codes are only applied when `isAnsiColorEnabled` is explicitly set to `true`. This prevents unwanted color codes in log files or non-terminal outputs. When disabled or not set, log levels are displayed as plain text.
+
+#### Example 4: Implement custom transport to handle logs your way
 
 The `transport` parameter allows you to implement custom logging behavior by providing your own logging functions. You can define handlers for different log levels (`debug`, `info`, `warn`, `error`, `trace`) to process log messages according to your needs.
 
@@ -358,7 +374,7 @@ The transport object should implement handlers for the log levels you want to cu
 
 For single transport you can use the `transport` parameter. For example:
 ```php
-$vwoClient3 = VWO::init([
+$vwoClient4 = VWO::init([
   'sdkKey' => '32-alpha-numeric-sdk-key',
   'logger' => [
     'transport' => [
@@ -373,7 +389,7 @@ $vwoClient3 = VWO::init([
 
 For multiple transports you can use the `transports` parameter. For example:
 ```php
-$vwoClient3 = VWO::init([
+$vwoClient5 = VWO::init([
   'sdkKey' => '32-alpha-numeric-sdk-key',
   'logger' => [
     'transports' => [
diff --git a/composer.json b/composer.json
@@ -1,7 +1,7 @@
 {
     "name": "vwo/vwo-fme-php-sdk",
 
-    "version": "1.16.0",
+    "version": "1.17.0",
     "keywords": ["vwo", "fme", "sdk"],
     "license": "Apache-2.0",
     "authors": [{
diff --git a/src/Constants/Constants.php b/src/Constants/Constants.php
@@ -40,7 +40,7 @@ class Constants {
     const DEFAULT_EVENTS_PER_REQUEST = 100;
     const SDK_NAME = 'vwo-fme-php-sdk';
 
-    const SDK_VERSION = '1.16.0';
+    const SDK_VERSION = '1.17.0';
     const AP = 'server';
 
     const SETTINGS = 'settings';
diff --git a/src/Packages/Logger/LogMessageBuilder.php b/src/Packages/Logger/LogMessageBuilder.php
@@ -56,16 +56,20 @@ public function getFormattedLevel($level): string {
             'WHITE' => "\x1b[30m",
             'YELLOW' => "\x1b[33m"
         ];
-
-        $logLevelColorInfoEnum = [
-            LogLevelEnum::TRACE => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['WHITE']}{$upperCaseLevel}{$ansiColorEnum['RESET']}",
-            LogLevelEnum::DEBUG => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['LIGHTBLUE']}{$upperCaseLevel} {$ansiColorEnum['RESET']}",
-            LogLevelEnum::INFO => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['CYAN']}{$upperCaseLevel}  {$ansiColorEnum['RESET']}",
-            LogLevelEnum::WARN => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['YELLOW']}{$upperCaseLevel}  {$ansiColorEnum['RESET']}",
-            LogLevelEnum::ERROR => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['RED']}{$upperCaseLevel} {$ansiColorEnum['RESET']}"
-        ];
-
-        return $logLevelColorInfoEnum[$level] ?? "{$ansiColorEnum['BOLD']}{$ansiColorEnum['RED']}INVALID{$ansiColorEnum['RESET']}";
+        // if loggerConfig.isAnsiColorEnabled is true, then use the formatted level, otherwise return the level as it is
+        if(isset($this->loggerConfig['isAnsiColorEnabled']) && $this->loggerConfig['isAnsiColorEnabled']) {
+            $logLevelColorInfoEnum = [
+                LogLevelEnum::TRACE => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['WHITE']}{$upperCaseLevel}{$ansiColorEnum['RESET']}",
+                LogLevelEnum::DEBUG => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['LIGHTBLUE']}{$upperCaseLevel} {$ansiColorEnum['RESET']}",
+                LogLevelEnum::INFO => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['CYAN']}{$upperCaseLevel}  {$ansiColorEnum['RESET']}",
+                LogLevelEnum::WARN => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['YELLOW']}{$upperCaseLevel}  {$ansiColorEnum['RESET']}",
+                LogLevelEnum::ERROR => "{$ansiColorEnum['BOLD']}{$ansiColorEnum['RED']}{$upperCaseLevel} {$ansiColorEnum['RESET']}"
+            ];
+    
+            return $logLevelColorInfoEnum[$level] ?? "{$ansiColorEnum['BOLD']}{$ansiColorEnum['RED']}INVALID{$ansiColorEnum['RESET']}";
+        } else {
+            return $upperCaseLevel;
+        }
     }
 
     public function getFormattedDateTime(): string {
diff --git a/src/Packages/NetworkLayer/Client/NetworkClient.php b/src/Packages/NetworkLayer/Client/NetworkClient.php
@@ -59,7 +59,7 @@ private function shouldUseCurl($networkOptions)
         return false;
     }
 
-    private function makeCurlRequest($url, $method, $headers, $body = null, $timeout = 5, $retryConfig = [])
+    private function makeCurlRequest($url, $method, $headers, $body = null, $timeout = 5000, $retryConfig = [])
     {
         // Merge with defaults to ensure all keys are present
         $retryConfig = array_merge(Constants::DEFAULT_RETRY_CONFIG, $retryConfig);
@@ -81,7 +81,7 @@ private function makeCurlRequest($url, $method, $headers, $body = null, $timeout
             curl_setopt_array($ch, [
                 CURLOPT_URL => $url,
                 CURLOPT_RETURNTRANSFER => true,
-                CURLOPT_TIMEOUT => $timeout,
+                CURLOPT_TIMEOUT_MS => $timeout,
                 CURLOPT_CUSTOMREQUEST => $method,
                 CURLOPT_SSL_VERIFYPEER => false,
                 CURLOPT_SSL_VERIFYHOST => false,
@@ -274,7 +274,7 @@ public function GET($request)
                 'GET',
                 $networkOptions['headers'],
                 null,
-                $networkOptions['timeout'] / 1000,
+                $networkOptions['timeout'],
                 $this->retryConfig
             );
             $rawResponse = $curlResponse['body'];
@@ -310,7 +310,7 @@ public function POST($request)
                     'POST',
                     $networkOptions['headers'],
                     $networkOptions['body'],
-                    $networkOptions['timeout'] / 1000,
+                    $networkOptions['timeout'],
                     $this->retryConfig
                 );
                 
diff --git a/src/Services/SettingsService.php b/src/Services/SettingsService.php
@@ -56,8 +56,8 @@ public function __construct($options, $logManager) {
         // Assigning values to properties
         $this->sdkKey = $options['sdkKey'];
         $this->accountId = $options['accountId'];
-        $this->expiry = isset($options['settings']['expiry']) ? $options['settings']['expiry'] : Constants::SETTINGS_EXPIRY;
-        $this->networkTimeout = isset($options['settings']['timeout']) ? $options['settings']['timeout'] : Constants::SETTINGS_TIMEOUT;
+        $this->expiry = isset($options['settingsConfig']['expiry']) ? $options['settingsConfig']['expiry'] : Constants::SETTINGS_EXPIRY;
+        $this->networkTimeout = isset($options['settingsConfig']['timeout']) ? $options['settingsConfig']['timeout'] : Constants::SETTINGS_TIMEOUT;
 
         // Parsing URL if provided
         if (isset($options['gatewayService']['url'])) {
diff --git a/src/Utils/LogMessageBuilder.php b/src/Utils/LogMessageBuilder.php

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "vwo/vwo-fme-php-sdk",`
`3`	`3`
`4`		`- "version": "1.16.0",`
	`4`	`+ "version": "1.17.0",`
`5`	`5`	`"keywords": ["vwo", "fme", "sdk"],`
`6`	`6`	`"license": "Apache-2.0",`
`7`	`7`	`"authors": [{`