Remove posix_getpwuid and compare only userid

Signed-off-by: hoellen <dev@hoellen.eu>
This commit is contained in:
hoellen 2020-10-03 16:32:49 +02:00
parent 0c89954488
commit 563f1318cd
2 changed files with 13 additions and 12 deletions

View File

@ -64,14 +64,14 @@ try {
echo "The posix extensions are required - see http://php.net/manual/en/book.posix.php" . PHP_EOL; echo "The posix extensions are required - see http://php.net/manual/en/book.posix.php" . PHP_EOL;
exit(1); exit(1);
} }
$user = posix_getpwuid(posix_getuid()); $user = posix_getuid();
$configUser = posix_getpwuid(fileowner(OC::$configDir . 'config.php')); $configUser = fileowner(OC::$configDir . 'config.php');
if ($user['name'] !== $configUser['name']) { if ($user !== $configUser) {
echo "Console has to be executed with the user that owns the file config/config.php" . PHP_EOL; echo "Console has to be executed with the user that owns the file config/config.php" . PHP_EOL;
echo "Current user: " . $user['name'] . PHP_EOL; echo "Current user id: " . $user . PHP_EOL;
echo "Owner of config.php: " . $configUser['name'] . PHP_EOL; echo "Owner id of config.php: " . $configUser . PHP_EOL;
echo "Try adding 'sudo -u " . $configUser['name'] . " ' to the beginning of the command (without the single quotes)" . 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['name'] . "' to the docker 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); exit(1);
} }

View File

@ -94,15 +94,16 @@ try {
exit(1); exit(1);
} }
$user = posix_getpwuid(posix_getuid()); $user = posix_getuid();
$configUser = posix_getpwuid(fileowner(OC::$configDir . 'config.php')); $configUser = fileowner(OC::$configDir . 'config.php');
if ($user['name'] !== $configUser['name']) { if ($user !== $configUser) {
echo "Console has to be executed with the user that owns the file config/config.php" . PHP_EOL; echo "Console has to be executed with the user that owns the file config/config.php" . PHP_EOL;
echo "Current user: " . $user['name'] . PHP_EOL; echo "Current user id: " . $user . PHP_EOL;
echo "Owner of config.php: " . $configUser['name'] . PHP_EOL; echo "Owner id of config.php: " . $configUser . PHP_EOL;
exit(1); exit(1);
} }
// We call Nextcloud from the CLI (aka cron) // We call Nextcloud from the CLI (aka cron)
if ($appMode !== 'cron') { if ($appMode !== 'cron') {
$config->setAppValue('core', 'backgroundjobs_mode', 'cron'); $config->setAppValue('core', 'backgroundjobs_mode', 'cron');