diff options
author | Simon L. <szaimen@e.mail.de> | 2024-05-14 12:09:15 +0200 |
---|---|---|
committer | backportbot[bot] <backportbot[bot]@users.noreply.github.com> | 2024-05-15 09:31:35 +0000 |
commit | 8068ad5b7130a97e407ff2c7b7caeb835f4a94fc (patch) | |
tree | db5ddb5ec066f19f99e86cf81eeec487b0ad18f2 | |
parent | cb63ff82c1050b4702715965d8eb549f1001c28d (diff) | |
download | nextcloud-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.php | 27 | ||||
-rw-r--r-- | cron.php | 8 |
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; @@ -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); } |