From 403ff688f8e811fc2ca1671718408a84857fe278 Mon Sep 17 00:00:00 2001
From: Pablo Zmdl <pablo@nextcloud.com>
Date: Tue, 16 Jun 2026 08:18:19 +0200
Subject: [PATCH] Fix fetching logs on podman initially

Podman doesn't like an empty `since` URL parameter.

Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
---
 php/src/Docker/DockerActionManager.php | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/php/src/Docker/DockerActionManager.php b/php/src/Docker/DockerActionManager.php
index a4adfcf1..173e4d5f 100644
--- a/php/src/Docker/DockerActionManager.php
+++ b/php/src/Docker/DockerActionManager.php
@@ -169,12 +169,11 @@ readonly class DockerActionManager {
     }
 
     public function GetLogs(string $id, string $since = ''): string {
-        $url = $this->BuildApiUrl(
-            sprintf(
-                'containers/%s/logs?stdout=true&stderr=true&timestamps=true&since=%s',
-                urlencode($id),
-                $since
-            ));
+        $path = sprintf('containers/%s/logs?stdout=true&stderr=true&timestamps=true', urlencode($id));
+        if ($since !== '') {
+            $path .= sprintf('&since=%s', $since);
+        }
+        $url = $this->BuildApiUrl($path);
         $responseBody = (string)$this->sendHttpRequest('GET', $url)->getBody();
 
         $response = "";