Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | fix(Storage): Fix method docs inheritancefix/storage/method-docs-inheritance | provokateurin | 2024-09-17 | 10 | -155/+16 |
| | | | | Signed-off-by: provokateurin <kate@provokateurin.de> | ||||
* | fix(Files): Handle getOwner() returning falsefix/storage/get-owner-false | provokateurin | 2024-09-17 | 3 | -10/+51 |
| | | | | Signed-off-by: provokateurin <kate@provokateurin.de> | ||||
* | fix(Storage): Document getOwner() can return false | provokateurin | 2024-09-17 | 7 | -41/+8 |
| | | | | 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-interface | Côme Chilliet | 2024-09-16 | 1 | -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 IStorage | Côme Chilliet | 2024-09-16 | 4 | -34/+0 |
| | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com> | ||||
* | chore: psalm-suppress legacy code weirdness | Côme Chilliet | 2024-09-16 | 1 | -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 Chilliet | 2024-09-16 | 6 | -15/+71 |
| | | | | | | Storage interface Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com> | ||||
* | fix: Fix other uses of removed Storage interface | Côme Chilliet | 2024-09-16 | 2 | -84/+8 |
| | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com> | ||||
* | chore: improve hash_file php usage in Local Storageartonge/fix/hash_return_type | John Molakvoæ | 2024-09-16 | 1 | -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 null | Louis Chemineau | 2024-09-16 | 2 | -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> | ||||
* | Merge pull request #47852 from nextcloud/sharding-code-fixes | John Molakvoæ | 2024-09-16 | 1 | -1/+1 |
|\ | |||||
| * | fix: misc code fixes around db shardingsharding-code-fixes | Robin Appelman | 2024-09-09 | 1 | -1/+1 |
| | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> | ||||
* | | Merge pull request #48016 from nextcloud/fix/s3configtrait/proxy-false | Kate | 2024-09-15 | 1 | -1/+1 |
|\ \ | |||||
| * | | fix(S3ConfigTrait): Allow proxy field to take falsefix/s3configtrait/proxy-false | provokateurin | 2024-09-15 | 1 | -1/+1 |
| | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de> | ||||
* | | | fix(Storage\Local): Do not call getSourcePath() on SplFileInfofix/storage-local/get-source-path-spl-file-info | provokateurin | 2024-09-15 | 1 | -1/+1 |
|/ / | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de> | ||||
* | | fix(files): Adjust Cache::searchQuery() parameter name to match interface | provokateurin | 2024-09-13 | 2 | -4/+4 |
| | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de> | ||||
* | | Merge pull request #47853 from nextcloud/clearify-swift-exception | Robin Appelman | 2024-09-10 | 1 | -4/+4 |
|\ \ | | | | | | | fix: make swift connect exception message more informative | ||||
| * | | fix: make swift connect exception message more informative | Robin Appelman | 2024-09-09 | 1 | -4/+4 |
| |/ | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> | ||||
* / | fix: Only write once to template instead of create/copy | Julius Knorr | 2024-09-10 | 1 | -3/+3 |
|/ | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net> | ||||
* | fix(files): Never return a null ETag in DAV | provokateurin | 2024-09-09 | 3 | -10/+3 |
| | | | | Signed-off-by: provokateurin <kate@provokateurin.de> | ||||
* | chore(storage): refactor some code portions | Git'Fellow | 2024-09-06 | 1 | -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> | ||||
* | Merge pull request #47784 from nextcloud/fix/file-info | Ferdinand Thiessen | 2024-09-06 | 1 | -9/+5 |
|\ | | | | | fix(FileInfo): correctly implement `\ArrayAccess::offsetGet` | ||||
| * | fix(FileInfo): correctly implement `\ArrayAccess::offsetGet` | Ferdinand Thiessen | 2024-09-06 | 1 | -9/+5 |
| | | | | | | | | | | | | | | We are targetting PHP 8.1+, since PHP 8.0 `offsetGet` has the `mixed` types for `offset` and return value. So this adds the types to correctly implement the interface. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de> | ||||
* | | fix(files): Check if the target path is a descendant of the shared folder path | Git'Fellow | 2024-09-05 | 1 | -1/+2 |
|/ | | | | | | | | | | | | | | | | | | 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 standard | Anna Larch | 2024-09-05 | 1 | -1/+1 |
| | | | | Signed-off-by: Anna Larch <anna@nextcloud.com> | ||||
* | Merge pull request #47417 from nextcloud/fix/files/create-mountpoint-parents | Kate | 2024-09-04 | 1 | -0/+9 |
|\ | |||||
| * | fix(files): Create non-existent parents of mountpoints | provokateurin | 2024-08-22 | 1 | -0/+9 |
| | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de> | ||||
* | | Merge pull request #47546 from nextcloud/fix/files-view-error-messages | Andy Scherzinger | 2024-08-29 | 1 | -3/+3 |
|\ \ | | | | | | | fix: Adjust filename validation messages | ||||
| * | | fix: Adjust filename validation messages | Ferdinand Thiessen | 2024-08-27 | 1 | -3/+3 |
| | | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de> | ||||
* | | | fix: Do not fail to get internal path on NonExistingFile/NonExistingFolder | Julius Härtl | 2024-08-29 | 2 | -0/+16 |
| | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net> | ||||
* | | | fix: Also validate parent path in `verifyPath` | Ferdinand Thiessen | 2024-08-28 | 2 | -3/+17 |
| | | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de> | ||||
* | | | fix: Allow read-only filename validation to allow reading files | Ferdinand Thiessen | 2024-08-28 | 1 | -1/+12 |
| | | | | | | | | | | | | | | | | | | Needed to read files with the "Windows compatibility" feature. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de> | ||||
* | | | fix: Renaming does not need update but delete permissions | Ferdinand Thiessen | 2024-08-28 | 1 | -5/+8 |
| | | | | | | | | | | | | | | | | | | | | | 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 recommendations | Louis Chemineau | 2024-08-28 | 1 | -9/+9 |
| | | | | | | | | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me> | ||||
* | | | fix: hint storage id in more places | Robin Appelman | 2024-08-28 | 1 | -2/+5 |
| | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> | ||||
* | | | feat: implement distributing partitioned queries over multiple shards | Robin Appelman | 2024-08-28 | 1 | -4/+92 |
|/ / | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> | ||||
* | | Merge pull request #46013 from ↵ | Julius Härtl | 2024-08-26 | 1 | -0/+25 |
|\ \ | | | | | | | | | | | | | nextcloud/obj-store-move-from-storage-preserve-fileid fix: write object to the correct urn when moving from another storage to object store | ||||
| * | | fix: write object to the correct urn when moving from another storage to ↵ | Robin Appelman | 2024-08-23 | 1 | -0/+25 |
| |/ | | | | | | | | | | | object store Signed-off-by: Robin Appelman <robin@icewind.nl> | ||||
* / | style: update codestyle for coding-standard 1.2.3 | Daniel Kesselberg | 2024-08-25 | 34 | -86/+86 |
|/ | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de> | ||||
* | fix: `FilenameValidator::isForbidden` should only check forbidden files | Ferdinand Thiessen | 2024-08-21 | 1 | -9/+18 |
| | | | | | | And not forbidden basenames as this is used for different purposes. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de> | ||||
* | fix: use mountpoint from storage to find the encryption keys | Robin Appelman | 2024-08-20 | 1 | -3/+3 |
| | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> | ||||
* | feat: store the mountpoint of storages in the mount options | Robin Appelman | 2024-08-20 | 1 | -2/+3 |
| | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> | ||||
* | Merge branch 'master' into jr-readdir-false-false | John Molakvoæ | 2024-08-16 | 123 | -3642/+3320 |
|\ | | | | | Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com> | ||||
| * | Merge pull request #47044 from nextcloud/fix/accept-several-mounts-in-encryption | Andy Scherzinger | 2024-08-07 | 1 | -1/+1 |
| |\ | | | | | | | fix(encryption): Fix mountpoint check to accept if several are found | ||||
| | * | fix(encryption): Fix mountpoint check to accept if several are found | Côme Chilliet | 2024-08-05 | 1 | -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: ensure array returned from getMountsForFileId is continious | Robin Appelman | 2024-08-02 | 1 | -2/+2 |
| | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> | ||||
| * | | fix(files): Fix incorrect keys by reindexing | Christopher Ng | 2024-08-01 | 1 | -2/+2 |
| | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com> | ||||
| * | | Merge pull request #46379 from nextcloud/fix/folder-search-owner | Ferdinand Thiessen | 2024-07-30 | 1 | -1/+24 |
| |\ \ | | |/ | |/| | fix: `OCP\Files\Node\Folder::search` was not setting the owner | ||||
| | * | fix: `OCP\Files\Node\Folder::search` was not setting the owner | Ferdinand Thiessen | 2024-07-11 | 1 | -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> | ||||
| * | | chore: fix some comments | withbest | 2024-07-29 | 1 | -1/+1 |
| | | | | | | | | | | | | Signed-off-by: withbest <seekseat@outlook.com> |