Log exceptions when creating share

Because the exceptions don't always contain a useful message for the UI,
but also because in some cases we need to find out what went wrong.

In some setups, a ShareNotFoundException might happen during creation
when we try to re-read the just written share. Usually related to Galera
Cluster where node syncing wait is not enabled.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

diff --git a/apps/files_sharing/lib/Controller/ShareAPIController.php b/apps/files_sharing/lib/Controller/ShareAPIController.php
index 0a1ad84f81612c9b75b85b80e74046d49b9b26e8..a051f9141ff05f8ce277abd071f5f45da5207633 100644 (file)
--- a/apps/files_sharing/lib/Controller/ShareAPIController.php
+++ b/apps/files_sharing/lib/Controller/ShareAPIController.php
@@ -631,9 +631,11 @@ class ShareAPIController extends OCSController {
                try {
                        $share = $this->shareManager->createShare($share);
                } catch (GenericShareException $e) {
+                       \OC::$server->getLogger()->logException($e);
                        $code = $e->getCode() === 0 ? 403 : $e->getCode();
                        throw new OCSException($e->getHint(), $code);
                } catch (\Exception $e) {
+                       \OC::$server->getLogger()->logException($e);
                        throw new OCSForbiddenException($e->getMessage(), $e);
                }