aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* fix(l10n): Update translations from TransifexHEADmasterNextcloud bot2 hours76-74/+380
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* Merge pull request #53728 from nextcloud/rector-phpunit10Robin Appelman10 hours479-2888/+1513
|\ | | | | chore/rector: make phpunit dataproviders phpunit 12 compatible
| * chore: run rectorrector-phpunit10Robin Appelman30 hours478-2887/+1508
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * chore: add rector DataProviderAnnotationToAttributeRectorRobin Appelman30 hours1-0/+2
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * chore: add phpunit StaticDataProviderClassMethodRector rector ruleRobin Appelman30 hours1-1/+3
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #53738 from nextcloud/leftybournes/fix/syslogKent Delante25 hours1-2/+6
|\ \ | | | | | | fix(syslog): open syslog channel on write
| * | fix(syslog): open syslog channel on writeKent Delante25 hours1-2/+6
|/ / | | | | | | | | | | | | This fixes a bug where only one tag gets used when multiple tags have been configured (e.g. different tags for 'syslog_tag' and 'syslog_tag_audit') Signed-off-by: Kent Delante <kent.delante@proton.me>
* / fix(l10n): Update translations from TransifexNextcloud bot26 hours128-618/+636
|/ | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* Merge pull request #53662 from nextcloud/feat/search-in-filesFerdinand Thiessen31 hours152-282/+1265
|\ | | | | feat(files): allow to proper search in files
| * chore: compile assetsfeat/search-in-filesFerdinand Thiessen33 hours126-174/+299
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * fix(files): do not redirect router when navigating between viewsFerdinand Thiessen33 hours1-8/+10
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * test: add e2e tests for files searchFerdinand Thiessen33 hours3-4/+232
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * fix(files): reload file list on search updateFerdinand Thiessen33 hours1-2/+17
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * feat(files): add search scope toggle and logicFerdinand Thiessen33 hours4-56/+149
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * feat(files): allow hidden viewsFerdinand Thiessen33 hours1-2/+6
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * feat(files): add `search` viewFerdinand Thiessen33 hours7-4/+308
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * refactor(files): adjust filename filter to use eventsFerdinand Thiessen33 hours2-2/+18
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * feat(files): add `search` store to handle all search related stateFerdinand Thiessen33 hours3-1/+178
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * fix(files): also use `open-in-files` for the search viewFerdinand Thiessen33 hours3-10/+12
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * feat(files): add `getDirectoryByPath` to files storeFerdinand Thiessen33 hours3-17/+25
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * fix(files): log aborted navigation as debug levelFerdinand Thiessen33 hours1-0/+9
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * fix(files_sharing): scope file action id to prevent conflictsFerdinand Thiessen33 hours1-2/+2
|/ | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(l10n): Update translations from TransifexNextcloud bot35 hours158-1432/+26876
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* Merge pull request #53625 from ↵Joas Schilling36 hours1341-2113/+3232
|\ | | | | | | | | nextcloud/dependabot/composer/vendor-bin/cs-fixer/nextcloud/coding-standard-1.4.0 chore(deps): bump nextcloud/coding-standard from 1.3.2 to 1.4.0 in /vendor-bin/cs-fixer
| * chore: Add coding standard refactor commit to .git-blame-ignore-revsCôme Chilliet36 hours1-0/+2
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * chore: apply new CSFixer rulesFerdinand Thiessen36 hours1339-2092/+3200
| | | | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de> # Conflicts: # apps/settings/lib/SetupChecks/PhpOpcacheSetup.php
| * chore(deps): bump nextcloud/coding-standard in /vendor-bin/cs-fixerdependabot[bot]36 hours1-21/+30
|/ | | | | | | | | | | | | | | | Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 1.3.2 to 1.4.0. - [Release notes](https://github.com/nextcloud/coding-standard/releases) - [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/coding-standard/compare/v1.3.2...v1.4.0) --- updated-dependencies: - dependency-name: nextcloud/coding-standard dependency-version: 1.4.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
* Merge pull request #53671 from nextcloud/fix/read-only-share-downloadJohn Molakvoæ37 hours20-74/+233
|\
| * fix: invalid bool returnfix/read-only-share-downloadJohn Molakvoæ7 days1-1/+0
| | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * feat(files_sharing): allow viewing files with download disabledskjnldsv7 days20-74/+234
| | | | | | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* | Merge pull request #53735 from nextcloud/bugfix/noid/document-hide-downloadJoas Schilling38 hours2-0/+7
|\ \ | | | | | | fix(ROS): Document hide-download option on files
| * | fix(ROS): Document hide-download option on filesbugfix/noid/document-hide-downloadJoas Schilling2 days2-0/+7
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #53429 from nextcloud/enh/opcache-checksFerdinand Thiessen38 hours1-14/+12
|\ \ \ | | | | | | | | feat(settings): simplify OPcache checks
| * | | feat(settings): simplify OPcache checksenh/opcache-checksMichaIng8 days1-14/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | | Merge pull request #53648 from nextcloud/rename-hooks-webhookFerdinand Thiessen38 hours1-4/+12
|\ \ \ \ | | | | | | | | | | fix: don't try to get fileid for non exising nodes when serializing events file
| * | | | fix: don't try to get fileid for non exising nodes when serializing events filerename-hooks-webhookRobin Appelman5 days1-4/+12
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | | Merge pull request #52976 from nextcloud/perf/realpath-custom-propKate39 hours1-0/+5
|\ \ \ \ \
| * | | | | perf: Skip photos app related property in custom propertiesperf/realpath-custom-propJulius Knorr2025-05-201-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | | | | | Merge pull request #53741 from ↵Stephan Orbaugh40 hours3-7/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/chore/public/user-mount-events-union-type chore(public): Remove unused union type for user mount events
| * | | | | | chore(public): Remove unused union type for user mount eventschore/public/user-mount-events-union-typeprovokateurin42 hours3-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | | | | | Merge pull request #53304 from nextcloud/fix/files_sharing/hide-own-resharesStephan Orbaugh40 hours5-109/+58
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | fix(files_sharing): Hide own reshares
| * | | | | | fix(files_sharing): Hide own resharesfix/files_sharing/hide-own-resharesprovokateurin44 hours1-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
| * | | | | | fix(files): Always transfer incoming shares when doing ownership transfer to ↵provokateurin44 hours4-95/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | prevent broken reshares Signed-off-by: provokateurin <kate@provokateurin.de>
| * | | | | | fix(files): Limit transferring incoming shares to the selected pathprovokateurin44 hours1-15/+23
|/ / / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | | | | fix(l10n): Update translations from TransifexNextcloud bot2 days106-122/+624
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | | | | | Merge pull request #53476 from invario/remove-x-xss-protectionJoas Schilling2 days8-20/+1
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Fix: Remove X-XSS-Protection use, check and recommendation
| * | | | | Merge branch 'master' into remove-x-xss-protectionJoas Schilling2 days2314-77292/+19750
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #52981 from nextcloud/perf/dav-preload-search-tagsArthur Schiwon2 days4-14/+52
|\ \ \ \ \ \ | | | | | | | | | | | | | | perf(dav): Preload dav search with tags/favorites
| * | | | | | perf(dav): Preload dav search with tags/favoritesperf/dav-preload-search-tagsJulius Knorr5 days4-14/+52
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | | | | | fix(l10n): Update translations from TransifexNextcloud bot3 days98-0/+98
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>