upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-06-11 01:30:50 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 79

Merge pull request #60544 from nextcloud/local-external-block-alias

Browse source 
fix: improve check if external storage backend is local
This commit is contained in:
Stephan Orbaugh 2026-05-20 16:03:29 +02:00 committed by GitHub
commit 988d8225be
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 17 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
apps/files_external/lib/Controller/GlobalStoragesController.php
View file

@ -77,16 +77,6 @@ class GlobalStoragesController extends StoragesController {
?array $applicableGroups,
?int $priority,
): DataResponse {
$canCreateNewLocalStorage = $this->config->getSystemValue('files_external_allow_create_new_local', true);
if (!$canCreateNewLocalStorage && $backend === 'local') {
return new DataResponse(
[
'message' => $this->l10n->t('Forbidden to manage local mounts')
],
Http::STATUS_FORBIDDEN
);
}
$newStorage = $this->createStorage(
$mountPoint,
$backend,

3
apps/files_external/lib/Controller/StoragesController.php
View file

@ -9,6 +9,7 @@ namespace OCA\Files_External\Controller;
use OCA\Files_External\Lib\Auth\AuthMechanism;
use OCA\Files_External\Lib\Backend\Backend;
use OCA\Files_External\Lib\Backend\Local;
use OCA\Files_External\Lib\InsufficientDataForMeaningfulAnswerException;
use OCA\Files_External\Lib\StorageConfig;
use OCA\Files_External\NotFoundException;
@ -78,7 +79,7 @@ abstract class StoragesController extends Controller {
?int $priority = null,
) {
$canCreateNewLocalStorage = $this->config->getSystemValue('files_external_allow_create_new_local', true);
if (!$canCreateNewLocalStorage && $backend === 'local') {
if (!$canCreateNewLocalStorage && $this->backendService->getBackend($backend) instanceof Local) {
return new DataResponse(
[
'message' => $this->l10n->t('Forbidden to manage local mounts')

9
apps/files_external/lib/Controller/UserStoragesController.php
View file

@ -102,15 +102,6 @@ class UserStoragesController extends StoragesController {
array $backendOptions,
?array $mountOptions,
): DataResponse {
$canCreateNewLocalStorage = $this->config->getSystemValue('files_external_allow_create_new_local', true);
if (!$canCreateNewLocalStorage && $backend === 'local') {
return new DataResponse(
[
'message' => $this->l10n->t('Forbidden to manage local mounts')
],
Http::STATUS_FORBIDDEN
);
}
$newStorage = $this->createStorage(
$mountPoint,
$backend,

2
apps/files_external/tests/Controller/GlobalStoragesControllerTest.php
View file

@ -50,7 +50,7 @@ class GlobalStoragesControllerTest extends StoragesControllerTestCase {
$session,
$this->createMock(IGroupManager::class),
$config,
$this->createMock(BackendService::class),
$this->backendService,
);
}

13
apps/files_external/tests/Controller/StoragesControllerTestCase.php
View file

@ -13,9 +13,11 @@ use OCA\Files_External\Controller\UserStoragesController;
use OCA\Files_External\Lib\Auth\AuthMechanism;
use OCA\Files_External\Lib\Auth\NullMechanism;
use OCA\Files_External\Lib\Backend\Backend;
use OCA\Files_External\Lib\Backend\Local;
use OCA\Files_External\Lib\Backend\SMB;
use OCA\Files_External\Lib\StorageConfig;
use OCA\Files_External\NotFoundException;
use OCA\Files_External\Service\BackendService;
use OCA\Files_External\Service\GlobalStoragesService;
use OCA\Files_External\Service\UserStoragesService;
use OCP\AppFramework\Http;
@ -24,9 +26,20 @@ use PHPUnit\Framework\MockObject\MockObject;
abstract class StoragesControllerTestCase extends \Test\TestCase {
protected GlobalStoragesController|UserStoragesController $controller;
protected GlobalStoragesService|UserStoragesService|MockObject $service;
protected BackendService|MockObject $backendService;
protected function setUp(): void {
parent::setUp();
$this->backendService = $this->createMock(BackendService::class);
$this->backendService->method('getBackend')
->willReturnCallback(function ($identifier) {
if ($identifier === 'local') {
return $this->createMock(Local::class);
} else {
return $this->createMock(Backend::class);
}
});
}
protected function tearDown(): void {

2
apps/files_external/tests/Controller/UserStoragesControllerTest.php
View file

@ -59,7 +59,7 @@ class UserStoragesControllerTest extends StoragesControllerTestCase {
$session,
$this->createMock(IGroupManager::class),
$config,
$this->createMock(BackendService::class),
$this->backendService,
);
}