Fix guzzle connect timeout parameter index

configcat · Jul 9, 2021 · 7b9ba53 · 7b9ba53
1 parent 115e34f
commit 7b9ba53
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/ConfigFetcher.php b/src/ConfigFetcher.php
@@ -77,7 +77,7 @@ public function __construct($sdkKey, LoggerInterface $logger, array $options = [
             ? $options['request-options']
             : [];
 
-        if (!isset($additionalOptions['connect-timeout'])) {
+        if (!isset($additionalOptions['connect_timeout'])) {
             $additionalOptions['connect_timeout'] = 10;
         }
 

diff --git a/tests/ConfigFetcherTest.php b/tests/ConfigFetcherTest.php
@@ -89,18 +89,18 @@ public function testConstructDefaults()
         $fetcher = new ConfigFetcher("api", new NullLogger());
         $options = $fetcher->getRequestOptions();
 
-        $this->assertEquals(10, $options['connect-timeout']);
+        $this->assertEquals(10, $options['connect_timeout']);
         $this->assertEquals(30, $options['timeout']);
         $this->assertArrayHasKey("headers", $options);
     }
 
     public function testConstructConnectTimeoutOption()
     {
         $fetcher = new ConfigFetcher("api", new NullLogger(), ['request-options' => [
-            'connect-timeout' => 5
+            'connect_timeout' => 5
         ]]);
         $options = $fetcher->getRequestOptions();
-        $this->assertEquals(5, $options['connect-timeout']);
+        $this->assertEquals(5, $options['connect_timeout']);
     }
 
     public function testConstructRequestTimeoutOption()