aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/Files/Storage
Commit message (Collapse)AuthorAgeFilesLines
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-193-8/+8
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(Storage): Fix method docs inheritancefix/storage/method-docs-inheritanceprovokateurin2024-09-179-151/+16
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(Storage): Document getOwner() can return falseprovokateurin2024-09-175-31/+6
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: Check storage is an instance of the correct class instead of ↵fix/remove-references-to-deprected-storage-interfaceCôme Chilliet2024-09-161-6/+9
| | | | | | psalm-suppress Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: Remove deprecated Storage::search method instead of moving it to IStorageCôme Chilliet2024-09-164-34/+0
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: psalm-suppress legacy code weirdnessCôme Chilliet2024-09-161-2/+12
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: Fix psalm issues, put back private versions of getter in private ↵Côme Chilliet2024-09-164-9/+63
| | | | | | Storage interface Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: Fix other uses of removed Storage interfaceCôme Chilliet2024-09-162-84/+8
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: improve hash_file php usage in Local Storageartonge/fix/hash_return_typeJohn Molakvoæ2024-09-161-8/+1
| | | | Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
* fix(files): Ensure that the hash method does not return nullLouis Chemineau2024-09-162-2/+10
| | | | | | | | To match https://github.com/nextcloud/server/blob/beececf66068f57c416225efcde9b44ce5c2e835/lib/private/Files/View.php#L1050 - Fix https://github.com/nextcloud/server/issues/44110 Signed-off-by: Louis Chemineau <louis@chmn.me>
* fix(Storage\Local): Do not call getSourcePath() on SplFileInfofix/storage-local/get-source-path-spl-file-infoprovokateurin2024-09-151-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(files): Never return a null ETag in DAVprovokateurin2024-09-093-10/+3
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(storage): refactor some code portionsGit'Fellow2024-09-061-30/+16
| | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> chore: revert portion Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* fix: Also validate parent path in `verifyPath`Ferdinand Thiessen2024-08-281-1/+13
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* style: update codestyle for coding-standard 1.2.3Daniel Kesselberg2024-08-257-24/+24
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* fix: use mountpoint from storage to find the encryption keysRobin Appelman2024-08-201-3/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge branch 'master' into jr-readdir-false-falseJohn Molakvoæ2024-08-1621-683/+412
|\ | | | | Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * fix(encryption): Fix mountpoint check to accept if several are foundCôme Chilliet2024-08-051-1/+1
| | | | | | | | | | | | | | There is no strong requirement to have only one mount for a given storage id. Also the error in this case would be misleading. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * chore: fix some commentswithbest2024-07-291-1/+1
| | | | | | | | Signed-off-by: withbest <seekseat@outlook.com>
| * fix: Release directory iterator and thereby its potential directory lock ↵Dennis Verspuij2024-07-271-0/+1
| | | | | | | | | | | | prior to deleting a directory, to avoid e.g. "Text file busy" error with VirtualBox shared folder storage Signed-off-by: Dennis Verspuij <6680484+dennisverspuij@users.noreply.github.com>
| * refactor: Migrate filename validation logic from `Storage` to ↵Ferdinand Thiessen2024-07-161-64/+12
| | | | | | | | | | | | `FilenameValidator` Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * Merge branch 'master' into refactor/OC-Server-getHTTPClientServiceJohn Molakvoæ2024-05-3021-596/+374
| |\ | | | | | | Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| | * chore: Add SPDX headerAndy Scherzinger2024-05-2421-513/+81
| | | | | | | | | | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
| | * fix: Fix newly spotted psalm issues, add exhaustive typed magic properties ↵Côme Chilliet2024-04-301-0/+1
| | | | | | | | | | | | | | | | | | for LDAP classes Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| | * fix(storage): Fix DAV storage as false is never returnedJoas Schilling2024-04-221-6/+0
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| | * fix(CI): Suppress psalm UndefinedClassJoas Schilling2024-04-181-0/+1
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| | * fix(files): Also skip cross storage move with access controlJoas Schilling2024-04-171-0/+2
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| | * Merge pull request #44294 from nextcloud/fix/storage-mount-ownerJulius Härtl2024-04-092-0/+17
| | |\ | | | | | | | | fix: Pass the mountpoint target user to storages without owner
| | | * fix: Pass the mountpoint target user to storages without ownerJulius Härtl2024-04-092-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Storages that do not have a dedicated owner (e.g. groupfolders, external storages) currently always assume the current session user as the owner. This leads to several issues when there is no user session but a node is obtained through a user folder. In order to have the correct user available we need to pass the user that is used to setup a mountpoint along to the storage layer as we generally assume that an owner is available for those. Signed-off-by: Julius Härtl <jus@bitgrid.net>
| | * | fix: use proper jailed patch in watcherRobin Appelman2024-04-081-4/+3
| | |/ | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| | * fix: Apply new coding standard to all filesCôme Chilliet2024-04-026-14/+14
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| | * fix: log error when default getWrapperStorage would return nullRobin Appelman2024-03-201-0/+9
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| | * fix: avoid scanning a non existing directoryBenjamin Gaussorgues2024-03-141-1/+5
| | | | | | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
| | * fix: add some recrusive detection/preventionRobin Appelman2024-03-081-0/+11
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| | * fix: Enforce forbidden filename characters on backendFerdinand Thiessen2024-02-281-5/+7
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| | * fix: Throw instead of yielding nothing when listing local directoriesJulius Härtl2024-02-231-0/+6
| | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
| | * use OCP\ServerRobin Appelman2024-02-121-1/+2
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| | * improve di performance for cacheRobin Appelman2024-02-092-9/+12
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| | * dav: fix wrong decoding of pluses in URLsLorenz Brun2024-01-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PHP's urldecode function does not decode URLs according to RFC 3986, but according to the HTML 4.01 query parameter encoding. This results in pluses being wrongly decoded to spaces even though they should not be decoded at all. Use rawurldecode instead, which follows RFC 3986 properly. This fixes an issue where files on DAV shares containing pluses were incorrectly decoded to spaces. Fixes: #15849 Signed-off-by: Lorenz Brun <lorenz@brun.one>
| | * chore: apply changes from Nextcloud coding standards 1.1.1Joas Schilling2023-11-232-2/+2
| | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
| | * Merge pull request #40935 from nextcloud/local-case-insensitiveBenjamin Gaussorgues2023-11-151-1/+26
| | |\ | | | | | | | | add some support for rename on case insensitive local filesystems
| | | * add some support for rename on case insensitive local filesystemsRobin Appelman2023-10-201-1/+26
| | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| | * | optimize cache jail creationRobin Appelman2023-10-191-4/+1
| | |/ | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| | * Convert isset ternary to null coalescing operatorHamid Dehnavi2023-09-282-3/+3
| | | | | | | | | | | | Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
| | * Merge pull request #40499 from nextcloud/known-mtime-wrapperRobin Appelman2023-09-211-0/+142
| | |\ | | | | | | | | add wrapper for external storage to ensure we don't get an mtime that is lower than we know it is
| | | * add wrapper to ensure we don't get an mtime that is lower than we know it isRobin Appelman2023-09-191-0/+142
| | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| | * | Merge pull request #39216 from shdehnavi/replace_substr_calls_in_lib_privateRobin Appelman2023-09-212-4/+4
| | |\ \ | | | |/ | | |/| Refactor "substr" calls in lib/private to improve code readability
| | | * Refactor "substr" calls to improve code readabilityHamid Dehnavi2023-07-072-4/+4
| | | | | | | | | | | | | | | | Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
| | * | fix: Only store unencrypted_size if path should be encryptedJulius Härtl2023-09-061-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
| | * | Merge pull request #39863 from nextcloud/sharing-mask-wrapperRobin Appelman2023-09-041-1/+2
| | |\ \ | | | | | | | | | | move share permission logic to storage wrapper