diff options
author | Carl Schwan <carl@carlschwan.eu> | 2022-11-23 16:04:42 +0100 |
---|---|---|
committer | backportbot-nextcloud[bot] <backportbot-nextcloud[bot]@users.noreply.github.com> | 2022-11-23 20:11:01 +0000 |
commit | c050ca9db4859e98cc1ef3cefe2f1c040a722736 (patch) | |
tree | 0320f07a8472079cf22a466fe11c545235d0b5ef /lib/private | |
parent | 222b8942af70a919dd3f2422927ed43e2fe19757 (diff) | |
download | nextcloud-server-c050ca9db4859e98cc1ef3cefe2f1c040a722736.tar.gz nextcloud-server-c050ca9db4859e98cc1ef3cefe2f1c040a722736.zip |
Add repair job that will ensure that secret and passwordsalt are set
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Diffstat (limited to 'lib/private')
-rw-r--r-- | lib/private/Repair.php | 2 | ||||
-rw-r--r-- | lib/private/Repair/NC25/AddMissingSecretJob.php | 64 |
2 files changed, 66 insertions, 0 deletions
diff --git a/lib/private/Repair.php b/lib/private/Repair.php index 5f4747721ca..9ca3ece6dd4 100644 --- a/lib/private/Repair.php +++ b/lib/private/Repair.php @@ -71,6 +71,7 @@ use OC\Repair\NC21\AddCheckForUserCertificatesJob; use OC\Repair\NC21\ValidatePhoneNumber; use OC\Repair\NC22\LookupServerSendCheck; use OC\Repair\NC24\AddTokenCleanupJob; +use OC\Repair\NC25\AddMissingSecretJob; use OC\Repair\OldGroupMembershipShares; use OC\Repair\Owncloud\CleanPreviews; use OC\Repair\Owncloud\DropAccountTermsTable; @@ -209,6 +210,7 @@ class Repair implements IOutput { \OCP\Server::get(LookupServerSendCheck::class), \OCP\Server::get(AddTokenCleanupJob::class), \OCP\Server::get(CleanUpAbandonedApps::class), + \OCP\Server::get(AddMissingSecretJob::class), ]; } diff --git a/lib/private/Repair/NC25/AddMissingSecretJob.php b/lib/private/Repair/NC25/AddMissingSecretJob.php new file mode 100644 index 00000000000..0aca640a943 --- /dev/null +++ b/lib/private/Repair/NC25/AddMissingSecretJob.php @@ -0,0 +1,64 @@ +<?php + +declare(strict_types=1); + +/** + * @copyright 2022 Carl Schwan <carl@carlschwan.eu> + * @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/>. + * + */ +namespace OC\Repair\NC25; + +use OC\Authentication\Token\TokenCleanupJob; +use OCP\HintException; +use OCP\IConfig; +use OCP\Migration\IOutput; +use OCP\Migration\IRepairStep; +use OCP\Security\ISecureRandom; + +class AddMissingSecretJob implements IRepairStep { + private IConfig $config; + private ISecureRandom $random; + + public function __construct(IConfig $config, ISecureRandom $random) { + $this->config = $config; + $this->random = $random; + } + + public function getName(): string { + return 'Add possibly missing system config'; + } + + public function run(IOutput $output): void { + $passwordSalt = $this->config->getSystemValue('passwordsalt', null); + if ($passwordSalt === null || $passwordSalt === '') { + try { + $this->config->setSystemValue('passwordsalt', $this->random->generate(30)); + } catch (HintException $e) { + $output->warning("passwordsalt is missing from your config.php and your config.php is read only. Please fix it manually."); + } + } + + $secret = $this->config->getSystemValue('secret', null); + if ($secret === null || $secret === '') { + try { + $this->config->setSystemValue('secret', $this->random->generate(48)); + } catch (HintException $e) { + $output->warning("secret is missing from your config.php and your config.php is read only. Please fix it manually."); + } + } + } +} |