From 7f22f3d42dcd72d344bd5c32961362d34600c1b6 Mon Sep 17 00:00:00 2001 From: provokateurin Date: Thu, 26 Sep 2024 09:38:41 +0200 Subject: [PATCH 1/3] fix(files_sharing): Parse OCM share permissions from OCM and not OCS prop Signed-off-by: provokateurin --- apps/files_sharing/lib/External/Storage.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/files_sharing/lib/External/Storage.php b/apps/files_sharing/lib/External/Storage.php index 7af22beda521b..020bbe2fbf9ef 100644 --- a/apps/files_sharing/lib/External/Storage.php +++ b/apps/files_sharing/lib/External/Storage.php @@ -362,7 +362,7 @@ public function getPermissions($path): int { $permissions = (int)$response['{http://open-collaboration-services.org/ns}share-permissions']; } elseif (isset($response['{http://open-cloud-mesh.org/ns}share-permissions'])) { // permissions provided by the OCM API - $permissions = $this->ocmPermissions2ncPermissions($response['{http://open-collaboration-services.org/ns}share-permissions'], $path); + $permissions = $this->ocmPermissions2ncPermissions($response['{http://open-cloud-mesh.org/ns}share-permissions'], $path); } elseif (isset($response['{http://owncloud.org/ns}permissions'])) { return $this->parsePermissions($response['{http://owncloud.org/ns}permissions']); } else { From 93b634f49795b32b6df8a59bbbf5297ec3e26d05 Mon Sep 17 00:00:00 2001 From: provokateurin Date: Thu, 26 Sep 2024 09:41:49 +0200 Subject: [PATCH 2/3] refactor(files_sharing): Make permissions prop checks less error prone Signed-off-by: provokateurin --- apps/files_sharing/lib/External/Storage.php | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/apps/files_sharing/lib/External/Storage.php b/apps/files_sharing/lib/External/Storage.php index 020bbe2fbf9ef..a85e32f7619ae 100644 --- a/apps/files_sharing/lib/External/Storage.php +++ b/apps/files_sharing/lib/External/Storage.php @@ -357,14 +357,17 @@ public function isSharable($path): bool { public function getPermissions($path): int { $response = $this->propfind($path); + $ocsPermissions = $response['{http://open-collaboration-services.org/ns}share-permissions'] ?? null; + $ocmPermissions = $response['{http://open-cloud-mesh.org/ns}share-permissions'] ?? null; + $ocPermissions = $response['{http://owncloud.org/ns}permissions'] ?? null; // old federated sharing permissions - if (isset($response['{http://open-collaboration-services.org/ns}share-permissions'])) { - $permissions = (int)$response['{http://open-collaboration-services.org/ns}share-permissions']; - } elseif (isset($response['{http://open-cloud-mesh.org/ns}share-permissions'])) { + if ($ocsPermissions !== null) { + $permissions = (int)$ocsPermissions; + } elseif ($ocmPermissions !== null) { // permissions provided by the OCM API - $permissions = $this->ocmPermissions2ncPermissions($response['{http://open-cloud-mesh.org/ns}share-permissions'], $path); - } elseif (isset($response['{http://owncloud.org/ns}permissions'])) { - return $this->parsePermissions($response['{http://owncloud.org/ns}permissions']); + $permissions = $this->ocmPermissions2ncPermissions($ocmPermissions, $path); + } elseif ($ocPermissions !== null) { + return $this->parsePermissions($ocPermissions); } else { // use default permission if remote server doesn't provide the share permissions $permissions = $this->getDefaultPermissions($path); From 35d2da42ec9dfcb292fb7358092304c6e77fdf17 Mon Sep 17 00:00:00 2001 From: provokateurin Date: Thu, 26 Sep 2024 14:35:25 +0200 Subject: [PATCH 3/3] fix(files_sharing): Check if propfind response is valid before accessing share permission props Signed-off-by: provokateurin --- apps/files_sharing/lib/External/Storage.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/apps/files_sharing/lib/External/Storage.php b/apps/files_sharing/lib/External/Storage.php index a85e32f7619ae..2a53bffb87e1c 100644 --- a/apps/files_sharing/lib/External/Storage.php +++ b/apps/files_sharing/lib/External/Storage.php @@ -357,6 +357,10 @@ public function isSharable($path): bool { public function getPermissions($path): int { $response = $this->propfind($path); + if ($response === false) { + return 0; + } + $ocsPermissions = $response['{http://open-collaboration-services.org/ns}share-permissions'] ?? null; $ocmPermissions = $response['{http://open-cloud-mesh.org/ns}share-permissions'] ?? null; $ocPermissions = $response['{http://owncloud.org/ns}permissions'] ?? null;