From 76c721f59153069369d151769d2d3624b859ed5c Mon Sep 17 00:00:00 2001 From: Maxence Lange Date: Wed, 17 Jun 2026 12:44:50 -0100 Subject: [PATCH] feat(federation): more logs on share failing Signed-off-by: Maxence Lange --- .../lib/Controller/RequestHandlerController.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/apps/cloud_federation_api/lib/Controller/RequestHandlerController.php b/apps/cloud_federation_api/lib/Controller/RequestHandlerController.php index b150bb43c13b9..1a824623d1b7a 100644 --- a/apps/cloud_federation_api/lib/Controller/RequestHandlerController.php +++ b/apps/cloud_federation_api/lib/Controller/RequestHandlerController.php @@ -157,6 +157,7 @@ public function addShare($shareWith, $name, $description, $providerId, $owner, $ Http::STATUS_BAD_REQUEST ); $response->throttle(); + $this->logger->info('user does not exist', ['user' => $shareWith]); return $response; } } @@ -171,6 +172,7 @@ public function addShare($shareWith, $name, $description, $providerId, $owner, $ Http::STATUS_BAD_REQUEST ); $response->throttle(); + $this->logger->info('group does not exist', ['user' => $shareWith]); return $response; } } @@ -208,11 +210,13 @@ public function addShare($shareWith, $name, $description, $providerId, $owner, $ } try { + $this->logger->info('share creation', ['sharedBy' => $sharedBy, 'sharedByDisplayName' => $sharedByDisplayName, 'ownerDisplayName' => $ownerDisplayName]); $provider = $this->cloudFederationProviderManager->getCloudFederationProvider($resourceType); $share = $this->factory->getCloudFederationShare($shareWith, $name, $description, $providerId, $owner, $ownerDisplayName, $sharedBy, $sharedByDisplayName, '', $shareType, $resourceType); $share->setProtocol($protocol); $provider->shareReceived($share); } catch (ProviderDoesNotExistsException|ProviderCouldNotAddShareException $e) { + $this->logger->error('issue with share provider', ['exception' => $e]); return new JSONResponse( ['message' => $e->getMessage()], Http::STATUS_NOT_IMPLEMENTED @@ -239,6 +243,7 @@ public function addShare($shareWith, $name, $description, $providerId, $owner, $ } } + $this->logger->info('share created', ['data' => $responseData]); return new JSONResponse($responseData, Http::STATUS_CREATED); }