Fix for info() failing with the new HTTP API format. Added test. (#65)

Co-authored-by: Steve Therrien <stherrien@onlineschool.ca>
centrifugal · Jun 27, 2024 · 88747d2 · 88747d2
1 parent 463e991
commit 88747d2
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/Client.php b/src/Client.php
@@ -432,7 +432,7 @@ private function request($method, $params)
         curl_setopt($ch, CURLOPT_HEADER, 0);
         curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
         curl_setopt($ch, CURLOPT_POST, true);
-        curl_setopt($ch, CURLOPT_POSTFIELDS, \json_encode($params));
+        curl_setopt($ch, CURLOPT_POSTFIELDS, \json_encode((object)$params));
         curl_setopt($ch, CURLOPT_HTTPHEADER, $this->getHeaders());
         curl_setopt($ch, CURLOPT_URL, $this->getUrl($method));
         $data = curl_exec($ch);

diff --git a/tests/ClientTest.php b/tests/ClientTest.php
@@ -70,4 +70,11 @@ public function testPublishNetworkError()
         $client = new \phpcent\Client("http://localhost:9000/api");
         $res = $client->publish('channel', ["message" => "Hello World"]);
     }
+
+    public function testInfo()
+    {
+        $res = $this->client->info();
+        $this->assertNotNull($res);
+        $this->assertTrue(is_array($res->result->nodes));
+    }
 }