summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimon L. <szaimen@e.mail.de>2024-05-14 12:09:15 +0200
committerbackportbot[bot] <backportbot[bot]@users.noreply.github.com>2024-05-15 09:31:35 +0000
commit8068ad5b7130a97e407ff2c7b7caeb835f4a94fc (patch)
treedb5ddb5ec066f19f99e86cf81eeec487b0ad18f2
parentcb63ff82c1050b4702715965d8eb549f1001c28d (diff)
downloadnextcloud-server-8068ad5b7130a97e407ff2c7b7caeb835f4a94fc.tar.gz
nextcloud-server-8068ad5b7130a97e407ff2c7b7caeb835f4a94fc.zip
Revert: "Check datadirectory owner, not config owner."
Signed-off-by: Simon L <szaimen@e.mail.de>
-rw-r--r--console.php27
-rw-r--r--cron.php8
2 files changed, 12 insertions, 23 deletions
diff --git a/console.php b/console.php
index 68647a874c4..96244615a87 100644
--- a/console.php
+++ b/console.php
@@ -65,28 +65,17 @@ try {
exit(1);
}
- // Check if the data directory is available and the server is installed
- $dataDirectory = $config->getSystemValueString('datadirectory', \OC::$SERVERROOT . '/data');
- if ($config->getSystemValueBool('installed', false) && !is_dir($dataDirectory)) {
- echo "Data directory (" . $dataDirectory . ") not found" . PHP_EOL;
+ $user = posix_getuid();
+ $configUser = fileowner(OC::$configDir . 'config.php');
+ if ($user !== $configUser) {
+ echo "Console has to be executed with the user that owns the file config/config.php" . PHP_EOL;
+ echo "Current user id: " . $user . PHP_EOL;
+ echo "Owner id of config.php: " . $configUser . PHP_EOL;
+ echo "Try adding 'sudo -u #" . $configUser . "' to the beginning of the command (without the single quotes)" . PHP_EOL;
+ echo "If running with 'docker exec' try adding the option '-u " . $configUser . "' to the docker command (without the single quotes)" . PHP_EOL;
exit(1);
}
- // Check if the user running the console is the same as the user that owns the data directory
- // If the data directory does not exist, the server is not setup yet and we can skip.
- if (is_dir($dataDirectory)) {
- $user = posix_getuid();
- $dataDirectoryUser = fileowner($dataDirectory);
- if ($user !== $dataDirectoryUser) {
- echo "Console has to be executed with the user that owns the data directory" . PHP_EOL;
- echo "Current user id: " . $user . PHP_EOL;
- echo "Owner id of the data directory: " . $dataDirectoryUser . PHP_EOL;
- echo "Try adding 'sudo -u #" . $dataDirectoryUser . "' to the beginning of the command (without the single quotes)" . PHP_EOL;
- echo "If running with 'docker exec' try adding the option '-u " . $dataDirectoryUser . "' to the docker command (without the single quotes)" . PHP_EOL;
- exit(1);
- }
- }
-
$oldWorkingDir = getcwd();
if ($oldWorkingDir === false) {
echo "This script can be run from the Nextcloud root directory only." . PHP_EOL;
diff --git a/cron.php b/cron.php
index e39c998ad5d..7cac2f44c58 100644
--- a/cron.php
+++ b/cron.php
@@ -115,11 +115,11 @@ try {
}
$user = posix_getuid();
- $dataDirectoryUser = fileowner($config->getSystemValueString('datadirectory', \OC::$SERVERROOT . '/data'));
- if ($user !== $dataDirectoryUser) {
- echo "Cron has to be executed with the user that owns the data directory" . PHP_EOL;
+ $configUser = fileowner(OC::$configDir . 'config.php');
+ if ($user !== $configUser) {
+ echo "Console has to be executed with the user that owns the file config/config.php" . PHP_EOL;
echo "Current user id: " . $user . PHP_EOL;
- echo "Owner id of the data directory: " . $dataDirectoryUser . PHP_EOL;
+ echo "Owner id of config.php: " . $configUser . PHP_EOL;
exit(1);
}