From 7cc9ba28a7e221e7e9f3723f3a6fb51ad57ea152 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Julius=20H=C3=A4rtl?= <jus@bitgrid.net>
Date: Tue, 31 Jan 2023 08:48:45 +0100
Subject: [PATCH] perf(federation): Only request root share info for checking
 availability
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Otherwise this would request a full recursive dirctory listing while the result is never being used

Signed-off-by: Julius Härtl <jus@bitgrid.net>
---
 .../lib/Controller/ShareInfoController.php       | 16 ++++++++++------
 apps/files_sharing/lib/External/Storage.php      |  6 +++---
 2 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/apps/files_sharing/lib/Controller/ShareInfoController.php b/apps/files_sharing/lib/Controller/ShareInfoController.php
index b090e6efcf1..2b8373af1cd 100644
--- a/apps/files_sharing/lib/Controller/ShareInfoController.php
+++ b/apps/files_sharing/lib/Controller/ShareInfoController.php
@@ -65,7 +65,7 @@ class ShareInfoController extends ApiController {
 	 * @param ?string $dir
 	 * @return JSONResponse
 	 */
-	public function info(string $t, ?string $password = null, ?string $dir = null) {
+	public function info(string $t, ?string $password = null, ?string $dir = null, int $depth = -1) {
 		try {
 			$share = $this->shareManager->getShareByToken($t);
 		} catch (ShareNotFound $e) {
@@ -96,28 +96,32 @@ class ShareInfoController extends ApiController {
 			}
 		}
 
-		return new JSONResponse($this->parseNode($node, $permissionMask));
+		return new JSONResponse($this->parseNode($node, $permissionMask, $depth));
 	}
 
-	private function parseNode(Node $node, int $permissionMask) {
+	private function parseNode(Node $node, int $permissionMask, int $depth) {
 		if ($node instanceof File) {
 			return $this->parseFile($node, $permissionMask);
 		}
-		return $this->parseFolder($node, $permissionMask);
+		return $this->parseFolder($node, $permissionMask, $depth);
 	}
 
 	private function parseFile(File $file, int $permissionMask) {
 		return $this->format($file, $permissionMask);
 	}
 
-	private function parseFolder(Folder $folder, int $permissionMask) {
+	private function parseFolder(Folder $folder, int $permissionMask, int $depth) {
 		$data = $this->format($folder, $permissionMask);
 
+		if ($depth === 0) {
+			return $data;
+		}
+
 		$data['children'] = [];
 
 		$nodes = $folder->getDirectoryListing();
 		foreach ($nodes as $node) {
-			$data['children'][] = $this->parseNode($node, $permissionMask);
+			$data['children'][] = $this->parseNode($node, $permissionMask, $depth <= -1 ? -1 : $depth - 1);
 		}
 
 		return $data;
diff --git a/apps/files_sharing/lib/External/Storage.php b/apps/files_sharing/lib/External/Storage.php
index 43568ea6a2b..f33334ca346 100644
--- a/apps/files_sharing/lib/External/Storage.php
+++ b/apps/files_sharing/lib/External/Storage.php
@@ -214,7 +214,7 @@ class Storage extends DAV implements ISharedStorage, IDisableEncryptionStorage,
 	public function checkStorageAvailability() {
 		// see if we can find out why the share is unavailable
 		try {
-			$this->getShareInfo();
+			$this->getShareInfo(0);
 		} catch (NotFoundException $e) {
 			// a 404 can either mean that the share no longer exists or there is no Nextcloud on the remote
 			if ($this->testRemote()) {
@@ -308,7 +308,7 @@ class Storage extends DAV implements ISharedStorage, IDisableEncryptionStorage,
 	 * @throws NotFoundException
 	 * @throws \Exception
 	 */
-	public function getShareInfo() {
+	public function getShareInfo(int $depth = -1) {
 		$remote = $this->getRemote();
 		$token = $this->getToken();
 		$password = $this->getPassword();
@@ -331,7 +331,7 @@ class Storage extends DAV implements ISharedStorage, IDisableEncryptionStorage,
 		$client = \OC::$server->getHTTPClientService()->newClient();
 		try {
 			$response = $client->post($url, [
-				'body' => ['password' => $password],
+				'body' => ['password' => $password, 'depth' => $depth],
 				'timeout' => 10,
 				'connect_timeout' => 10,
 			]);
-- 
2.39.5