aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/Files
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #47861 from nextcloud/backport/47860/stable28John Molakvoæ2024-09-151-3/+1
|\
| * fix: Only write once to template instead of create/copybackport/47860/stable28Julius Knorr2024-09-101-3/+1
| | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | fix(Storage\Local): Do not call getSourcePath() on SplFileInfobackport/48015/stable28provokateurin2024-09-151-1/+1
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix(files): Adjust Cache::searchQuery() parameter name to match interfacebackport/47954/stable28provokateurin2024-09-132-4/+4
|/ | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(files): Create non-existent parents of mountpointsprovokateurin2024-09-041-0/+9
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: Pass the mountpoint target user to storages without ownerJulius Härtl2024-09-032-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 mountpoint from storage to find the encryption keysRobin Appelman2024-08-201-3/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* feat: store the mountpoint of storages in the mount optionsRobin Appelman2024-08-201-2/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(encryption): Fix mountpoint check to accept if several are foundCôme Chilliet2024-08-071-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>
* fix: `OCP\Files\Node\Folder::search` was not setting the ownerFerdinand Thiessen2024-07-301-1/+24
| | | | | | 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: Do not return partial file info if we have a cache entryJulius Härtl2024-07-251-3/+4
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* fix: FileInfo from `View` should have the correct name of a mountpointFerdinand Thiessen2024-07-241-0/+4
| | | | | | | | | | If a mountpoint is returned from a `View` the name must match the view scoped name and not the global name. For example group folders have a name like `1` or `2` (the ID), but the name of the FileInfo returned fro mthe View should be the mount point name, like `First groupfolder` or similar. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(s3): Don't wait indefinitely for S3 to returnGit'Fellow2024-07-091-29/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: lint Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: use AwsException Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Throw on connection failure Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Wrap all in try catch block Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: use RequestTimeout error message Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> log: use OCP Server class Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Handle connect timeout only Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Handle errors more generically Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* Merge pull request #45839 from nextcloud/backport/45649/stable28Andy Scherzinger2024-06-131-1/+1
|\ | | | | [stable28] fix(ObjectStore): Fix regression when hostname is empty
| * fix(ObjectStore): handle empty S3 hostnameJosh2024-06-131-1/+1
| | | | | | | | | | | | | | | | Fixes #45637 The support for s3-accelerate added in #44496 introduced a regression in AWS S3 environments when `hostname` is blank (which is a valid configuration w/ AWS since the hostname gets auto-generated). Signed-off-by: Josh <josh.t.richards@gmail.com>
* | Merge pull request #45857 from nextcloud/backport/45172/stable28Andy Scherzinger2024-06-131-1/+5
|\ \ | | | | | | [stable28] feat: add option to disable scanner transactions
| * | feat: add option to disable scanner transactionsRobin Appelman2024-06-131-1/+5
| |/ | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* / fix: get child ids for folder in a separate query during moveRobin Appelman2024-06-131-4/+21
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(files): Use isRetryable to catch retryable exceptionsLouis Chemineau2024-06-121-2/+6
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* Merge pull request #45528 from nextcloud/backport/45014/stable28John Molakvoæ2024-06-071-0/+6
|\
| * Fix backportGit'Fellow2024-05-281-0/+5
| | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
| * fix: forbid moving a folder into a subfolder of itselfRobin Appelman2024-05-271-0/+1
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> [skip ci]
* | Merge pull request #45059 from nextcloud/backport/44496/stable28Josh2024-06-031-1/+16
|\ \ | | | | | | [stable28] feat: support "s3-accelerate" endpoint
| * | feat: add option to enable s3 accelerationRobin Appelman2024-05-101-1/+16
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Try create folder first and check if it exists afterGit'Fellow2024-05-301-1/+1
| | | | | | | | | | | | Co-authored-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com> Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | | fix(files): Try to create dir only if it not existsGit'Fellow2024-05-301-1/+1
| | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | | perf(db): Sort data for IN before chunkingChristoph Wurst2024-05-291-0/+3
| |/ |/| | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | fix(Files): Change how scanner diffs for changed metadataJosh2024-05-131-2/+47
|/ | | | | Fixes #43408 Signed-off-by: Josh <josh.t.richards@gmail.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>
* fix: use proper jailed patch in watcherRobin Appelman2024-04-102-4/+77
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* perf: Avoid updating the folder size if we know the size differenceJulius Härtl2024-04-052-15/+19
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* feat(files)): Add retry logic to cover deadlock situations when moving many ↵Louis Chemineau2024-03-281-6/+25
| | | | | | files Signed-off-by: Louis Chemineau <louis@chmn.me>
* Merge pull request #44320 from nextcloud/backport/39990/stable28Stephan Orbaugh2024-03-251-0/+11
|\ | | | | [stable28] add some recrusive detection/prevention
| * fix: add some recrusive detection/preventionRobin Appelman2024-03-221-0/+11
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | fix: log error when default getWrapperStorage would return nullRobin Appelman2024-03-221-0/+9
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #44285 from nextcloud/backport/44279/stable28John Molakvoæ2024-03-211-0/+7
|\
| * fix: ensure nested mount points are handled in the correct orderRobin Appelman2024-03-181-0/+7
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #44053 from nextcloud/backport/43086/stable28John Molakvoæ2024-03-191-1/+2
|\ \ | |/ |/|
| * fix: disable the internal s3sdk multi part copy logicRobin Appelman2024-03-071-1/+2
| | | | | | | | 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>
* | Refactoring getName functionhopleus2024-03-141-5/+3
| | | | | | | | Signed-off-by: hopleus <hopleus@gmail.com>
* | Fixes for getting the file name in the cases when the file name key is ↵hopleus2024-03-141-1/+5
|/ | | | | | received, but it is empty Signed-off-by: hopleus <hopleus@gmail.com>
* fix: Throw instead of yielding nothing when listing local directoriesJulius Härtl2024-02-261-0/+6
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Merge pull request #43391 from nextcloud/backport/43131/stable28Robin Appelman2024-02-091-1/+1
|\ | | | | [stable28] fix(files): Don't attempt to format a partial cache entry
| * fix(files): Don't attempt to format a partial cache entryChristoph Wurst2024-02-061-1/+1
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | continue background scanning different storages if one failsRobin Appelman2024-02-071-23/+27
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* dav: fix wrong decoding of pluses in URLsLorenz Brun2024-02-031-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>
* also updated parent etags when a changed etag is detected during scanningRobin Appelman2024-01-312-7/+30
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* dont reuse etag for folders marked explicitly unscannedRobin Appelman2024-01-311-1/+2
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>