sascha
/
nextcloud-server
mirror of https://github.com/nextcloud/server.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

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

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
pull/51420/head
Ferdinand Thiessen 2025-03-10 15:31:45 +07:00 committed by backportbot[bot]
parent c83233e8cf
commit e7793a3ea9
6 changed files with 12 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
apps/federatedfilesharing/lib/FederatedShareProvider.php
Unescape Escape View File

@ -1008,8 +1008,8 @@ class FederatedShareProvider implements IShareProvider {
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');
return $result === 'yes';
}

2
apps/federatedfilesharing/tests/FederatedShareProviderTest.php
Unescape Escape View File

@ -848,7 +848,7 @@ class FederatedShareProviderTest extends \Test\TestCase {
$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,

9
apps/files_sharing/lib/Controller/ShareesAPIController.php
Unescape Escape View File

@ -212,12 +212,9 @@ class ShareesAPIController extends OCSController {
$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);

2
apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php
Unescape Escape View File

@ -260,7 +260,7 @@ class ShareesAPIControllerTest extends TestCase {
->method('getAppValue')
->with($this->anything(), $this->anything(), $this->anything())
->willReturnMap([
['files_sharing', 'lookupServerEnabled', 'yes', 'yes'],
['files_sharing', 'lookupServerEnabled', 'no', 'yes'],
]);
$this->shareManager->expects($this->once())

2
lib/private/Collaboration/Collaborators/LookupPlugin.php
Unescape Escape View File

@ -54,7 +54,7 @@ class LookupPlugin implements ISearchPlugin {
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

8
tests/lib/Collaboration/Collaborators/LookupPluginTest.php
Unescape Escape View File

@ -89,7 +89,7 @@ class LookupPluginTest extends TestCase {
public function testSearchNoLookupServerURI() {
$this->config->expects($this->once())
->method('getAppValue')
->with('files_sharing', 'lookupServerEnabled', 'yes')
->with('files_sharing', 'lookupServerEnabled', 'no')
->willReturn('yes');
$this->config->expects($this->exactly(2))
->method('getSystemValueBool')
@ -118,7 +118,7 @@ class LookupPluginTest extends TestCase {
public function testSearchNoInternet() {
$this->config->expects($this->once())
->method('getAppValue')
->with('files_sharing', 'lookupServerEnabled', 'yes')
->with('files_sharing', 'lookupServerEnabled', 'no')
->willReturn('yes');
$this->config->expects($this->exactly(2))
->method('getSystemValueBool')
@ -154,7 +154,7 @@ class LookupPluginTest extends TestCase {
$this->config->expects($this->once())
->method('getAppValue')
->with('files_sharing', 'lookupServerEnabled', 'yes')
->with('files_sharing', 'lookupServerEnabled', 'no')
->willReturn('yes');
$this->config->expects($this->exactly(2))
->method('getSystemValueBool')
@ -214,7 +214,7 @@ class LookupPluginTest extends TestCase {
$this->config->expects($this->once())
->method('getAppValue')
->with('files_sharing', 'lookupServerEnabled', 'yes')
->with('files_sharing', 'lookupServerEnabled', 'no')
->willReturn($LookupEnabled ? 'yes' : 'no');
if ($GSEnabled || $LookupEnabled) {
$searchResult->expects($this->once())