nextcloud / nextcloud/nextcloudpi
Set NC loglevel via ncp
Nobody has claimed this yet.
- Dominant language
- Shell
- Stars
- 2.9k
- Forks
- 318
- PR merge metrics
- No merged PRs in 30d
Description
Description
Currently, there is an issue with Nextcloud's logs that are flooded with messages like:
appinfo/app.php is deprecated, use \\OCP\\AppFramework\\Bootstrap\\IBootstrap on the application class instead.
This is discussed here https://github.com/nextcloud/server/issues/23046
Especially in NextCloudPi I fear this means unnecessary wear and tear on the SD card. A solution would be to set the loglevel in config/config.php to something higher than 0 (which is debug logs, https://docs.nextcloud.com/server/15/admin_manual/configuration_server/logging_configuration.html).
This could be implemented as an entry under Config similar to nc-trusted-domains. Unfortunately I am not familiar with PHP, so I am not able to provide an example implementation.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the ncp Config implementation and the existing nc-trusted-domains entry, then inspect how it writes config/config.php. Determine how the loglevel value should be exposed through ncp and verify that running the command updates the Nextcloud configuration as intended.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php, shell
- Domain
- backend, cli
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 32/100