DockerActionManager: improve the logging

Signed-off-by: Simon L. <[email protected]>
nextcloud · Jan 20, 2025 · 72f91b4 · 72f91b4
1 parent 2992198
commit 72f91b4
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/php/src/Docker/DockerActionManager.php b/php/src/Docker/DockerActionManager.php
@@ -167,7 +167,8 @@ public function StartContainer(Container $container) : void {
         try {
             $this->guzzleClient->post($url);
         } catch (RequestException $e) {
-            throw new \Exception("Could not start container " . $container->GetIdentifier() . ": " . $e->getMessage());
+            error_log("Could not start container " . $container->GetIdentifier() . ": " . $e->getMessage());
+            throw $e;
         }
     }
 
@@ -588,7 +589,8 @@ public function CreateContainer(Container $container) : void {
                 ]
             );
         } catch (RequestException $e) {
-            throw new \Exception("Could not create container " . $container->GetIdentifier() . ": " . $e->getMessage());
+            error_log("Could not create container " . $container->GetIdentifier() . ": " . $e->getMessage());
+            throw $e;
         }
 
     }
@@ -883,7 +885,8 @@ private function ConnectContainerIdToNetwork(string $id, string $internalPort, s
             } catch (RequestException $e) {
                 // 409 is undocumented and gets thrown if the network already exists.
                 if ($e->getCode() !== 409) {
-                    throw new \Exception("Could not create the nextcloud-aio network: " . $e->getMessage());
+                    error_log("Could not create the nextcloud-aio network: " . $e->getMessage());
+                    throw $e;
                 }
             }
         }