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

Updater ignores additional config.php files #384

Open
PascalReintjens opened this issue Jul 25, 2021 · 5 comments · Fixed by #535
Open

Updater ignores additional config.php files #384

PascalReintjens opened this issue Jul 25, 2021 · 5 comments · Fixed by #535

Comments

@PascalReintjens
Copy link

As it is described in https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/config_sample_php_parameters.html#multiple-config-php-file, it is possible to use multiple config.php files.

Unfortunately, however, the updater.phar file seems to ignore the other config files, which causes the updater to not work properly on some otherwise smoothly running Nextcloud instances.

One problem I and many other Nextcloud users have as a result is the following:
There are hosting plans where the basic file path is different when a user accesses Nextcloud via web browser than when, for example, OCC and updater.phar are accessed via SSH. Then a dynamic adjustment of the datadirectory is necessary. This can not be done in the config.php, because this could be overwritten by Nextcloud at any time again to a fixed path when updates are applied or the admin changes other values in the settings and hence the main config file gets rebuilt. Therefore, many fix this by creating a separate config file, like this one at https://github.com/froonix/webhosting-mods/blob/master/httpdocs/nextcloud/nextcloud/config/data.config.php to overwrite the value of the config. This works fine except for the updater.

Users of such and other usecases probably wonder why occ works fine, but the updater.phar has problems and may not get the idea that multiple config files are supported for Nextcloud, but not for the updater. If you have read the documentation you don't expect such a behavior and I could imagine that this is responsible for some bug reports.

@PascalReintjens PascalReintjens changed the title Updater ignores additional config.php files Updater.phar ignores additional config.php files Jul 25, 2021
@PascalReintjens PascalReintjens changed the title Updater.phar ignores additional config.php files Updater ignores additional config.php files Jul 25, 2021
@mrschaal
Copy link

Also experiencing exactly the same issue on Nextcloud 22, hosted by netcup webhosting plans.

@Cr4zyJ0k3r
Copy link

I have the same issue on the same hoster.

@homberger
Copy link

I have the same issue on the same hoster.

Me too. Netcup Webhosting.

@come-nc
Copy link
Collaborator

come-nc commented Mar 18, 2024

Had to revert the PR, reopening this issue.

Ideas on how to proceed in #535 (comment)

@come-nc come-nc reopened this Mar 18, 2024
@MasterPuffin
Copy link

What helps in this case is modifying the config with the following

  'datadirectory' => php_sapi_name() === 'cli' ? '/httpdocs/nextcloud/data' : '/var/www/vhosts/{{instance}}.netcup.net/httpdocs/nextcloud/data',

This is however only a temporary fix and Nextcloud also overwrites this line with every update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants