fix(lookup-server): do not query data by default

Signed-off-by: Ferdinand Thiessen <[email protected]>
nextcloud · Mar 10, 2025 · 98cfdea · 98cfdea
1 parent f65d1bb
commit 98cfdea
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 10 deletions.
diff --git a/apps/federatedfilesharing/lib/FederatedShareProvider.php b/apps/federatedfilesharing/lib/FederatedShareProvider.php
@@ -998,8 +998,7 @@ public function isLookupServerQueriesEnabled(): bool {
 		if ($this->gsConfig->isGlobalScaleEnabled()) {
 			return true;
 		}
-		$result = $this->config->getAppValue('files_sharing', 'lookupServerEnabled', 'yes');
-		return $result === 'yes';
+		$result = $this->config->getAppValue('files_sharing', 'lookupServerEnabled', 'no');
 	}
 
 

diff --git a/apps/federatedfilesharing/tests/FederatedShareProviderTest.php b/apps/federatedfilesharing/tests/FederatedShareProviderTest.php
@@ -825,7 +825,7 @@ public function testIsLookupServerQueriesEnabled($gsEnabled, $isEnabled, $expect
 		$this->gsConfig->expects($this->once())->method('isGlobalScaleEnabled')
 			->willReturn($gsEnabled);
 		$this->config->expects($this->any())->method('getAppValue')
-			->with('files_sharing', 'lookupServerEnabled', 'yes')
+			->with('files_sharing', 'lookupServerEnabled', 'no')
 			->willReturn($isEnabled);
 
 		$this->assertSame($expected,

diff --git a/apps/files_sharing/lib/Controller/ShareesAPIController.php b/apps/files_sharing/lib/Controller/ShareesAPIController.php
@@ -176,12 +176,9 @@ public function search(string $search = '', ?string $itemType = null, int $page
 		$this->offset = $perPage * ($page - 1);
 
 		// In global scale mode we always search the loogup server
-		if ($this->config->getSystemValueBool('gs.enabled', false)) {
-			$lookup = true;
-			$this->result['lookupEnabled'] = true;
-		} else {
-			$this->result['lookupEnabled'] = $this->config->getAppValue('files_sharing', 'lookupServerEnabled', 'yes') === 'yes';
-		}
+		$lookup = $this->config->getSystemValueBool('gs.enabled', false)
+			|| $this->config->getAppValue('files_sharing', 'lookupServerEnabled', 'no') === 'yes';
+		$this->result['lookupEnabled'] = $lookup;
 
 		[$result, $hasMoreResults] = $this->collaboratorSearch->search($search, $shareTypes, $lookup, $this->limit, $this->offset);
 

diff --git a/lib/private/Collaboration/Collaborators/LookupPlugin.php b/lib/private/Collaboration/Collaborators/LookupPlugin.php
@@ -32,7 +32,7 @@ public function __construct(
 
 	public function search($search, $limit, $offset, ISearchResult $searchResult): bool {
 		$isGlobalScaleEnabled = $this->config->getSystemValueBool('gs.enabled', false);
-		$isLookupServerEnabled = $this->config->getAppValue('files_sharing', 'lookupServerEnabled', 'yes') === 'yes';
+		$isLookupServerEnabled = $this->config->getAppValue('files_sharing', 'lookupServerEnabled', 'no') === 'yes';
 		$hasInternetConnection = $this->config->getSystemValueBool('has_internet_connection', true);
 
 		// if case of Global Scale we always search the lookup server