Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RFE] Optionally disable filesystem ownership permissions check in console.php #24914

Closed
e-minguez opened this issue Dec 30, 2020 · 1 comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement

Comments

@e-minguez
Copy link

Is your feature request related to a problem? Please describe.
Related to nextcloud/docker#1344

The console.php file has a check to ensure the ownership... that fails if using nextcloud in containers with NFS as backend storage for /var/www/html with proper all_squash,anonuid=82,anongid=0 to match www-data:root ownership.

server/console.php

Lines 68 to 75 in 47b4efc

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);
}

Describe the solution you'd like
Some config.php setting to disable the check would be nice.

Describe alternatives you've considered
As a workaround, I've just modified the file to skip the exit(1) call

Additional context

@szaimen
Copy link
Contributor

szaimen commented Jun 22, 2021

Closing as duplicate of #13277

@szaimen szaimen closed this as completed Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement
Projects
None yet
Development

No branches or pull requests

2 participants