aboutsummaryrefslogtreecommitdiffstats
path: root/tests/lib/Files
Commit message (Collapse)AuthorAgeFilesLines
* fix(files): Check if the target path is a descendant of the shared folder pathGit'Fellow2024-09-051-1/+9
| | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: fix tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: add tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: tests
* chore: adjust code to adhere to coding standardAnna Larch2024-09-051-1/+1
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* Merge pull request #47417 from nextcloud/fix/files/create-mountpoint-parentsKate2024-09-041-0/+31
|\
| * fix(files): Create non-existent parents of mountpointsprovokateurin2024-08-221-0/+31
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | Merge pull request #47546 from nextcloud/fix/files-view-error-messagesAndy Scherzinger2024-08-291-1/+1
|\ \ | | | | | | fix: Adjust filename validation messages
| * | fix: Adjust filename validation messagesFerdinand Thiessen2024-08-271-1/+1
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | fix: Renaming does not need update but delete permissionsFerdinand Thiessen2024-08-281-3/+3
| | | | | | | | | | | | | | | | | | | | | Renaming is basically copy + delete (a move), so no need to update permissions. Especially if the node is in a invalid directory the node should be moveable but not editable. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | chore: Apply php:cs recommendationsLouis Chemineau2024-08-281-7/+7
| | | | | | | | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* | | feat: implement distributing partitioned queries over multiple shardsRobin Appelman2024-08-281-0/+17
|/ / | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* / style: update codestyle for coding-standard 1.2.3Daniel Kesselberg2024-08-2515-163/+163
|/ | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* ci: Skip flaky test on PHP 8.3Joas Schilling2024-08-211-0/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix: `FilenameValidator::isForbidden` should only check forbidden filesFerdinand Thiessen2024-08-211-17/+7
| | | | | | And not forbidden basenames as this is used for different purposes. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge branch 'master' into jr-readdir-false-falseJohn Molakvoæ2024-08-1674-953/+1672
|\ | | | | Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * Merge pull request #46379 from nextcloud/fix/folder-search-ownerFerdinand Thiessen2024-07-301-3/+11
| |\ | | | | | | fix: `OCP\Files\Node\Folder::search` was not setting the owner
| | * fix: `OCP\Files\Node\Folder::search` was not setting the ownerFerdinand Thiessen2024-07-111-3/+11
| | | | | | | | | | | | | | | | | | The owner was not set on the file info causing e.g. webdav searches to never return the known owner. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | fix(db): Promote the use of `getDatabaseProvider` to reduce the impage of ↵Joas Schilling2024-07-191-2/+2
| | | | | | | | | | | | | | | | | | removed upstream platforms Signed-off-by: Joas Schilling <coding@schilljs.com>
| * | fix: make batch propagator work with sharding restrictionsRobin Appelman2024-07-171-1/+1
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix: make usermountcache compatible with shardingRobin Appelman2024-07-171-22/+33
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix(dav): Verify target path in `setName` instead of source pathFerdinand Thiessen2024-07-161-2/+2
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | refactor: Migrate filename validation logic from `Storage` to ↵Ferdinand Thiessen2024-07-163-99/+107
| | | | | | | | | | | | | | | | | | `FilenameValidator` Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | feat: Add `forbidden_filename_basenames` config optionFerdinand Thiessen2024-07-151-27/+58
| |/ | | | | | | | | | | | | | | This allows to configure forbidden filenames (the full filename like `.htaccess`) and also forbidden basenames like `com0` where `com0`, `com0.txt` and `com0.tar.gz` will match. We need this as only using basenames was too restrictive and will cause problems on some systems when updating. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * feat: Add `IFilenameValidator` to have one consistent place for filename ↵Ferdinand Thiessen2024-07-091-0/+188
| | | | | | | | | | | | | | | | validation Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * chore: Add SPDX headerAndy Scherzinger2024-05-1373-690/+211
| | | | | | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
| * test: add test for jail watcherRobin Appelman2024-04-081-0/+30
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * fix: Apply new coding standard to all filesCôme Chilliet2024-04-023-3/+3
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * Merge pull request #40131 from summersab/refactor/OC-Server-getShareManagerJohn Molakvoæ2024-03-151-1/+2
| |\
| | * Merge branch 'master' into refactor/OC-Server-getShareManagerJohn Molakvoæ2024-03-1531-176/+1083
| | |\ | | | | | | | | Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| | * | Refactor `OC\Server::getShareManager`Andrew Summers2023-08-291-3/+4
| | | | | | | | | | | | | | | | Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com>
| * | | Refactor `OC\Server::getLockingProvider`Andrew Summers2024-03-151-3/+3
| | |/ | |/| | | | | | | Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com>
| * | refactor(mimeloader): modernize MimeTypeLoaderThomas Citharel2024-03-141-6/+5
| | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
| * | feat(files): copy live photosMaxence Lange2024-03-081-3/+6
| | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
| * | fix: clearify logic around getMountsForFileId filteringRobin Appelman2024-03-061-11/+11
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | Merge pull request #43471 from nextcloud/cache-path-by-idRobin Appelman2024-03-056-45/+92
| |\ \ | | | | | | | | Cache path by id
| | * | perf: cache path by id to speedup getting nodes by idRobin Appelman2024-03-046-45/+92
| | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | test: Additional tests for issue #35776Dariusz Olszewski2024-03-041-0/+146
| |/ / | | | | | | | | | Signed-off-by: Dariusz Olszewski <starypatyk@users.noreply.github.com>
| * | fix: Enforce forbidden filename characters on backendFerdinand Thiessen2024-02-281-3/+48
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | add some comments for the distributive operation and add another testRobin Appelman2024-02-161-0/+27
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | add extra flatten step to improve "or eq" -> "in" optimizationRobin Appelman2024-02-151-3/+3
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | add a search query step to split IN statements that are to large for ociRobin Appelman2024-02-151-0/+44
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | optimize query pattern used by storage filterRobin Appelman2024-02-155-0/+341
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | only get the path for the users cached mount info when we use itRobin Appelman2024-02-091-3/+5
| | | | | | | | | | | | | | | | | | most of the time we shouldn't need it so we can save joining on the filecache Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | make storage directory test be a bit less strict about mtimesRobin Appelman2024-02-071-2/+3
| | | | | | | | | | | | | | | | | | sometimes the clock ticks over during the test Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | refactor(objectstorage): cleanup typesThomas Citharel2024-02-021-2/+2
| | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
| * | add test for updating etag with unchanged mtime in child folderRobin Appelman2024-01-311-0/+25
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | add test for updating etag with unchanged mtimeRobin Appelman2024-01-311-0/+19
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix(tests): Add messages to assertions to explain what failedJoas Schilling2024-01-261-8/+8
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * | chore(tests): Migrate object storage unit tests from drone to GitHub ActionsFerdinand Thiessen2024-01-261-0/+4
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | fix(tests): Fix deprecation errors in S3 testsFerdinand Thiessen2024-01-261-1/+2
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | chore: apply changes from Nextcloud coding standards 1.1.1Joas Schilling2023-11-238-24/+24
| | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
| * | fix(ObjectStore): Make copying behavior consistent with local storageFerdinand Thiessen2023-11-171-0/+38
| | | | | | | | | | | | | | | | | | Drop file permissions on copy like we do on local storage. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>