search

nextcloud / updater

:arrows_counterclockwise: The updater app to keep your Nextcloud up-to-date
GNU Affero General Public License v3.0
45 stars 33 forks source link

Bug: Check for expected files fails when logfile is configured #482

Closed Yannik closed 1 year ago

Yannik commented 1 year ago

I have configured 'logfile' => 'nextcloud.log', in config/config.php.

Trying to update from 27.0.0 to 27.0.1, the upgrade fails during the Check for expected files phase:

Screenshot from 2023-07-26 12-07-27

Expected behavior: This should not occur, because the nextcloud.log file is actually expected to be there, as it is configured as logfile.

Yannik commented 1 year ago

Note that 'logfile' => 'nextcloud.log' exactly matches the example in the documentation.

joshtrichards commented 1 year ago

Hi @Yannik - Where is there? In the webrot? Are you sure that's your currently active nextcloud.log and not an old/stale nextcloud.log? When configured as you stated, the log file will be in your configured datadirectory (not webroot) and thus excluded from the expected files check.

kesselb commented 1 year ago

Why would you put nextcloud.log in the root directory :thinking:

Your bug report is valid from a technical point because we don't add the configured logfile automatically to the list of expected files.

I don't see a good reason why someone would place the logfile there and therefore close the issue.