aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_external/lib/Command/Notify.php
diff options
context:
space:
mode:
Diffstat (limited to 'apps/files_external/lib/Command/Notify.php')
-rw-r--r--apps/files_external/lib/Command/Notify.php162
1 files changed, 25 insertions, 137 deletions
diff --git a/apps/files_external/lib/Command/Notify.php b/apps/files_external/lib/Command/Notify.php
index 5013d88539b..a4e7edf0207 100644
--- a/apps/files_external/lib/Command/Notify.php
+++ b/apps/files_external/lib/Command/Notify.php
@@ -3,36 +3,12 @@
declare(strict_types=1);
/**
- * @copyright Copyright (c) 2016 Robin Appelman <robin@icewind.nl>
- *
- * @author Ari Selseng <ari@selseng.net>
- * @author Christoph Wurst <christoph@winzerhof-wurst.at>
- * @author Joas Schilling <coding@schilljs.com>
- * @author Robin Appelman <robin@icewind.nl>
- * @author Roeland Jago Douma <roeland@famdouma.nl>
- *
- * @license GNU AGPL version 3 or any later version
- *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License as
- * published by the Free Software Foundation, either version 3 of the
- * License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU Affero General Public License for more details.
- *
- * You should have received a copy of the GNU Affero General Public License
- * along with this program. If not, see <http://www.gnu.org/licenses/>.
- *
+ * SPDX-FileCopyrightText: 2016 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
*/
namespace OCA\Files_External\Command;
use Doctrine\DBAL\Exception\DriverException;
-use OC\Core\Command\Base;
-use OCA\Files_External\Lib\InsufficientDataForMeaningfulAnswerException;
-use OCA\Files_External\Lib\StorageConfig;
use OCA\Files_External\Service\GlobalStoragesService;
use OCP\DB\QueryBuilder\IQueryBuilder;
use OCP\Files\Notify\IChange;
@@ -40,7 +16,6 @@ use OCP\Files\Notify\INotifyHandler;
use OCP\Files\Notify\IRenameChange;
use OCP\Files\Storage\INotifyStorage;
use OCP\Files\Storage\IStorage;
-use OCP\Files\StorageNotAvailableException;
use OCP\IDBConnection;
use OCP\IUserManager;
use Psr\Log\LoggerInterface;
@@ -49,24 +24,14 @@ use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
-class Notify extends Base {
- private GlobalStoragesService $globalService;
- private IDBConnection $connection;
- private LoggerInterface $logger;
- /** @var IUserManager */
- private $userManager;
-
+class Notify extends StorageAuthBase {
public function __construct(
+ private IDBConnection $connection,
+ private LoggerInterface $logger,
GlobalStoragesService $globalService,
- IDBConnection $connection,
- LoggerInterface $logger,
- IUserManager $userManager
+ IUserManager $userManager,
) {
- parent::__construct();
- $this->globalService = $globalService;
- $this->connection = $connection;
- $this->logger = $logger;
- $this->userManager = $userManager;
+ parent::__construct($globalService, $userManager);
}
protected function configure(): void {
@@ -107,82 +72,15 @@ class Notify extends Base {
parent::configure();
}
- private function getUserOption(InputInterface $input): ?string {
- if ($input->getOption('user')) {
- return (string)$input->getOption('user');
- } elseif (isset($_ENV['NOTIFY_USER'])) {
- return $_ENV['NOTIFY_USER'];
- } elseif (isset($_SERVER['NOTIFY_USER'])) {
- return $_SERVER['NOTIFY_USER'];
- } else {
- return null;
- }
- }
-
- private function getPasswordOption(InputInterface $input): ?string {
- if ($input->getOption('password')) {
- return (string)$input->getOption('password');
- } elseif (isset($_ENV['NOTIFY_PASSWORD'])) {
- return $_ENV['NOTIFY_PASSWORD'];
- } elseif (isset($_SERVER['NOTIFY_PASSWORD'])) {
- return $_SERVER['NOTIFY_PASSWORD'];
- } else {
- return null;
- }
- }
-
protected function execute(InputInterface $input, OutputInterface $output): int {
- $mount = $this->globalService->getStorage($input->getArgument('mount_id'));
- if (is_null($mount)) {
- $output->writeln('<error>Mount not found</error>');
- return 1;
- }
- $noAuth = false;
-
- $userOption = $this->getUserOption($input);
- $passwordOption = $this->getPasswordOption($input);
-
- // if only the user is provided, we get the user object to pass along to the auth backend
- // this allows using saved user credentials
- $user = ($userOption && !$passwordOption) ? $this->userManager->get($userOption) : null;
-
- try {
- $authBackend = $mount->getAuthMechanism();
- $authBackend->manipulateStorageConfig($mount, $user);
- } catch (InsufficientDataForMeaningfulAnswerException $e) {
- $noAuth = true;
- } catch (StorageNotAvailableException $e) {
- $noAuth = true;
+ [$mount, $storage] = $this->createStorage($input, $output);
+ if ($storage === null) {
+ return self::FAILURE;
}
- if ($userOption) {
- $mount->setBackendOption('user', $userOption);
- }
- if ($passwordOption) {
- $mount->setBackendOption('password', $passwordOption);
- }
-
- try {
- $backend = $mount->getBackend();
- $backend->manipulateStorageConfig($mount, $user);
- } catch (InsufficientDataForMeaningfulAnswerException $e) {
- $noAuth = true;
- } catch (StorageNotAvailableException $e) {
- $noAuth = true;
- }
-
- try {
- $storage = $this->createStorage($mount);
- } catch (\Exception $e) {
- $output->writeln('<error>Error while trying to create storage</error>');
- if ($noAuth) {
- $output->writeln('<error>Username and/or password required</error>');
- }
- return 1;
- }
if (!$storage instanceof INotifyStorage) {
$output->writeln('<error>Mount of type "' . $mount->getBackend()->getText() . '" does not support active update notifications</error>');
- return 1;
+ return self::FAILURE;
}
$dryRun = $input->getOption('dry-run');
@@ -204,15 +102,10 @@ class Notify extends Base {
}
$this->markParentAsOutdated($mount->getId(), $change->getPath(), $output, $dryRun);
});
- return 0;
- }
-
- private function createStorage(StorageConfig $mount): IStorage {
- $class = $mount->getBackend()->getStorageClass();
- return new $class($mount->getBackendOptions());
+ return self::SUCCESS;
}
- private function markParentAsOutdated($mountId, $path, OutputInterface $output, bool $dryRun) {
+ private function markParentAsOutdated($mountId, $path, OutputInterface $output, bool $dryRun): void {
$parent = ltrim(dirname($path), '/');
if ($parent === '.') {
$parent = '';
@@ -253,22 +146,17 @@ class Notify extends Base {
}
}
- private function logUpdate(IChange $change, OutputInterface $output) {
- switch ($change->getType()) {
- case INotifyStorage::NOTIFY_ADDED:
- $text = 'added';
- break;
- case INotifyStorage::NOTIFY_MODIFIED:
- $text = 'modified';
- break;
- case INotifyStorage::NOTIFY_REMOVED:
- $text = 'removed';
- break;
- case INotifyStorage::NOTIFY_RENAMED:
- $text = 'renamed';
- break;
- default:
- return;
+ private function logUpdate(IChange $change, OutputInterface $output): void {
+ $text = match ($change->getType()) {
+ INotifyStorage::NOTIFY_ADDED => 'added',
+ INotifyStorage::NOTIFY_MODIFIED => 'modified',
+ INotifyStorage::NOTIFY_REMOVED => 'removed',
+ INotifyStorage::NOTIFY_RENAMED => 'renamed',
+ default => '',
+ };
+
+ if ($text === '') {
+ return;
}
$text .= ' ' . $change->getPath();
@@ -324,7 +212,7 @@ class Notify extends Base {
}
- private function selfTest(IStorage $storage, INotifyHandler $notifyHandler, OutputInterface $output) {
+ private function selfTest(IStorage $storage, INotifyHandler $notifyHandler, OutputInterface $output): void {
usleep(100 * 1000); //give time for the notify to start
if (!$storage->file_put_contents('/.nc_test_file.txt', 'test content')) {
$output->writeln("Failed to create test file for self-test");