| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | |
| | |
| | |
| | | |
Signed-off-by: Edward Ly <contact@edward.ly>
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
This allows to configure which view should be the default ("start view")
in the files app, currently either "all files" or "personal files".
But it might be extended to the new home view in the future.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| |
| |
| |
| | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| |
| |
| |
| | |
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
|
| |
| |
| |
| |
| |
| | |
Mocked server initial state was incomplete
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
| |
| |
| |
| | |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
| |
| |
| |
| |
| |
| |
| | |
Remove the hack which was needed until the view.hidden attribute was
public API.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
|\ \
| | |
| | | |
fix(encryption): Catch exceptions in encrypt-all command and continue
|
| | |
| | |
| | |
| | | |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
|\ \ \
| | | |
| | | | |
fix(profile): add max width for the user actions
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This prevents the profile to look distorted when the primary action is a
long email address.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
|\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | | |
nextcloud/fix/avoid-crashing-versions-listener-on-non-existing-file
fix(files_versions): Log error instead of crashing when event listeners get called on non-existing files
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
called on non-existing files
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
|\ \ \ \
| |_|/ /
|/| | | |
Fix TimedJob execution time to allow job execution exactly when scheduled
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
These jobs that were setting their interval to 0 were not really running
at every run of cron.php if the run was in the same second. To keep the
same behaior, I am updating those intervals to 1 second.
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
- default boundaries element is 'content-vue', which includes sidebar
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
|
|\ \ \
| | | |
| | | | |
fix(files_versions): Log failure to compute node path
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
To avoid hidden bugs, add some debug logging when a node path cannot be
computed from a received event, to have more information for debugging
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
| |_|/
|/| |
| | |
| | | |
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
|
|\ \ \
| | | |
| | | | |
chore/rector: make phpunit dataproviders phpunit 12 compatible
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Robin Appelman <robin@icewind.nl>
|
|/ / /
| | |
| | |
| | | |
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
|
| |/
|/|
| |
| |
| |
| |
| | |
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
# Conflicts:
# apps/settings/lib/SetupChecks/PhpOpcacheSetup.php
|
|\ \ |
|
| | |
| | |
| | |
| | | |
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
|
|\ \ \
| | | |
| | | | |
feat(settings): simplify OPcache checks
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
For the overall OPcache size check, we currently compare used memory with free memory. However, `opcache.memory_consumption` is split into `used_memory`, `free_memory` and `wasted_memory`. When cached files change on disk, old entries are not replaced or removed, but remain as wasted memory, until the cache is actually full, and if their percentage is above `opcache.max_wasted_percentage`, which is 5% by default. When this happens, the engine is restarted, resetting the cache completely, like a `opcache_reset()` call.
As long as we do not consider wasted cache, recommendations based on free memory can be false. To solve this, we could count wasted memory as free memory, if it is above `opcache.max_wasted_percentage`, as the engine will be restarted as soon as needed, freeing up this wasted space. On the other hand, wasted memory below the threshold permanently blocks the OPcache, which supports counting it as used memory. Depending on the situation, instead of raising OPcache size, it could be also advised to reduce `opcache.max_wasted_percentage`. But too frequent cache resets break its purpose as well.
In my opinion, the matter is too complex to consider wasted cache correctly, and do precise recommendations, but we should focus on reducing false positives instead. What we know for sure is: if the cache is full (`$status['cache_full'] === true`), and the limit for cached keys has not been reached, the OPcache was too small to maintain free space, with wasted memory below the configured threshold, where it consumes memory permanently. Recommending to raise the OPcache size in this case, is hence as accurate as it gets. Even if 5% wasted cache could be freed, 95% used memory is still above the previous threshold for the setup check warning. And if `opcache.max_wasted_percentage` is above 5%, then the admin must have decided to change the default, deciding that system memory consumption has lower priority than preventing OPcache engine restarts.
`cache_full` can be true as well if the limit for cached keys has been reached, hence we need to merge both checks. In this case `num_cached_keys` equals `max_cached_keys` exactly, hence it is easy to differentiale whether `opcache.max_accelerated_files` or `opcache.memory_consumption` needs to be raised to address the `cache_full` state.
In practice, this change relaxes the checks: the respective limit needs to be reached 100% instead of 90%, to trigger a warning, eliminating also false alarms if a large share of the cache is consumed by wasted memory, which would be automatically freed once cache is 100% full.
Additionally, the recommendation for raising `opcache.max_accelerated_files` now says "a value higher than `max_cached_keys`", instead of "higher than `opcache.max_accelerated_files`". The actual limit, reflected by `max_cached_keys` from `opcache_get_status()`, [is a next higher value from a set of prime numbers](https://www.php.net/manual/en/opcache.configuration.php#ini.opcache.max-accelerated-files). E.g. if `opcache.max_accelerated_files` is set to 10,000 (PHP default), the effective limit is 16,229 OPcache keys. Recommending "higher than 10000" could hence lead to a settings change without effect. For an effective change, the new value needs to be "higher than 16229" instead, which is what the setup check will show in this situation, with this change applied.
Signed-off-by: MichaIng <micha@dietpi.com>
|
|\ \ \ \ |
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Julius Knorr <jus@bitgrid.net>
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: provokateurin <kate@provokateurin.de>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
prevent broken reshares
Signed-off-by: provokateurin <kate@provokateurin.de>
|