feat: return log responses as strings without JSON parsing

Author: loks0n

src/KubernetesCluster.php

@@ -183,7 +183,7 @@ public function runOperation(Operation|string $operation, string $path, string|n
             Operation::APPLY => $this->applyPath($path, $payload, $query),
             Operation::JSON_PATCH => $this->jsonPatchPath($path, $payload, $query),
             Operation::JSON_MERGE_PATCH => $this->jsonMergePatchPath($path, $payload, $query),
-            default => $this->makeRequest($operation->httpMethod(), $path, $payload, $query),
+            default => $this->makeRequest($operation, $path, $payload, $query),
         };
     }
 
@@ -351,7 +351,7 @@ protected function execPath(
         array $query = ['pretty' => 1, 'stdin' => 1, 'stdout' => 1, 'stderr' => 1, 'tty' => 1]
     ): mixed {
         try {
-            return $this->makeRequest(Operation::EXEC->httpMethod(), $path, '', $query);
+            return $this->makeRequest(Operation::EXEC, $path, '', $query);
         } catch (KubernetesAPIException $e) {
             $payload = $e->getPayload();
 
@@ -380,7 +380,7 @@ protected function attachPath(
         array $query = ['pretty' => 1, 'stdin' => 1, 'stdout' => 1, 'stderr' => 1, 'tty' => 1]
     ): mixed {
         try {
-            return $this->makeRequest(Operation::ATTACH->httpMethod(), $path, '', $query);
+            return $this->makeRequest(Operation::ATTACH, $path, '', $query);
         } catch (KubernetesAPIException $e) {
             $payload = $e->getPayload();
 
@@ -411,7 +411,7 @@ protected function applyPath(string $path, string $payload, array $query = ['pre
             ],
         ];
 
-        return $this->makeRequest(Operation::APPLY->httpMethod(), $path, $payload, $query, $options);
+        return $this->makeRequest(Operation::APPLY, $path, $payload, $query, $options);
     }
 
     /**
@@ -428,7 +428,7 @@ protected function jsonPatchPath(string $path, string $payload, array $query = [
             ],
         ];
 
-        return $this->makeRequest(Operation::JSON_PATCH->httpMethod(), $path, $payload, $query, $options);
+        return $this->makeRequest(Operation::JSON_PATCH, $path, $payload, $query, $options);
     }
 
     /**
@@ -445,7 +445,7 @@ protected function jsonMergePatchPath(string $path, string $payload, array $quer
             ],
         ];
 
-        return $this->makeRequest(Operation::JSON_MERGE_PATCH->httpMethod(), $path, $payload, $query, $options);
+        return $this->makeRequest(Operation::JSON_MERGE_PATCH, $path, $payload, $query, $options);
     }
 
     /**

src/Traits/Cluster/MakesHttpCalls.php

@@ -5,6 +5,7 @@
 use GuzzleHttp\Client;
 use GuzzleHttp\Exception\ClientException;
 use GuzzleHttp\RequestOptions;
+use RenokiCo\PhpK8s\Enums\Operation;
 use RenokiCo\PhpK8s\Exceptions\KubernetesAPIException;
 use RenokiCo\PhpK8s\ResourcesList;
 
@@ -146,18 +147,23 @@ public function call(string $method, string $path, string $payload = '', array $
     }
 
     /**
-     * Call the API with the specified method and path.
+     * Call the API with the specified operation and path.
      *
      * @return mixed
      *
      * @throws \RenokiCo\PhpK8s\Exceptions\KubernetesAPIException
      */
-    protected function makeRequest(string $method, string $path, string $payload = '', array $query = ['pretty' => 1], array $options = [])
+    protected function makeRequest(Operation $operation, string $path, string $payload = '', array $query = ['pretty' => 1], array $options = [])
     {
         $resourceClass = $this->resourceClass;
 
+        $method = $operation->httpMethod();
         $response = $this->call($method, $path, $payload, $query, $options);
 
+        if ($operation === Operation::LOG) {
+            return (string) $response->getBody();
+        }
+
         $json = @json_decode($response->getBody(), true);
 
         // If the output is not JSONable, return the response itself.