- changed the "servers" option for configuration array (no need for MEMCACHED_POOL env variable)

- fixes the unit test

Author: kodeart

Handler/MemcachedHandler.php

@@ -66,14 +66,10 @@ public function gc($maxLifetime): bool
 
     private function configuration(SessionConfiguration $settings): array
     {
-        if (!$servers = $settings->get('servers')) {
-            $servers = json_decode(getenv('MEMCACHED_POOL'), true) ?? [['127.0.0.1', 11211]];
-        }
-
         return [
-            'servers' => $servers,
-            'id'      => $settings->get('id', $settings->get('name', ini_get('session.name'))),
-            'options' => $settings->get('options', []) + [
+            'servers' => (array)$settings->get('servers', [['127.0.0.1', 11211]]),
+            'id'      => (string)$settings->get('id', $settings->get('name', ini_get('session.name'))),
+            'options' => (array)$settings->get('options', []) + [
                     \Memcached::OPT_PREFIX_KEY => $settings->prefix ?? 'session.'
                 ]
         ];

Tests/Handler/MemcachedHandlerTest.php

@@ -26,15 +26,15 @@ protected function setUp()
                 'name'                    => 'test',
                 'save_handler'            => 'memcached',
                 'expire_at_browser_close' => false,
+                'use_cookies'    => false,
+                'cache_limiter'  => '',
+                'gc_maxlifetime' => 60,
 
+                'servers' => defined('CI') ? [['127.0.0.1', 11211]] : [['memcached', 11211]],
                 'options' => [
                     \Memcached::OPT_DISTRIBUTION => null,
                     \Memcached::OPT_PREFIX_KEY   => 'sess.'
                 ],
-
-                'use_cookies'    => false,
-                'cache_limiter'  => '',
-                'gc_maxlifetime' => 60,
             ]
         ]);