summaryrefslogtreecommitdiffstats
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* [tx-robot] updated from transifexNextcloud bot2021-11-0618-0/+60
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* Update lib/private/Files/Storage/Wrapper/Availability.php add void Vincent Petry2021-11-051-1/+1
| | | | | Signed-off-by: Vincent Petry <vincent@nextcloud.com> Co-authored-by: Carl Schwan <carl@carlschwan.eu>
* Return false in hasUpdated when storage is not availableVincent Petry2021-11-051-4/+10
| | | | | | | | | | Technically, saying that a storage has no updates when it's not available is correct. This makes it possible to retrieve the cache entry for the mount point and also to list and remove unavailable federated shares. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* [tx-robot] updated from transifexNextcloud bot2021-11-0522-22/+0
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* Fix psalmJohn Molakvoæ2021-11-041-2/+2
| | | | Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
* Merge pull request #29482 from nextcloud/feat/revamp-profile-visibility-sectionv23.0.0beta3John Molakvoæ2021-11-041-52/+141
|\
| * Refine profile backend for frontend visibility requirementsChristopher Ng2021-11-041-44/+115
| | | | | | | | | | | | | | | | | | - Provide metadata - Dynamic displayId - Add appId - Filter out unused parameter config properties from the existing profile config Signed-off-by: Christopher Ng <chrng8@gmail.com>
| * Better function descriptionsChristopher Ng2021-11-031-2/+4
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
| * Add additional check on action registrationsChristopher Ng2021-11-031-8/+24
| | | | | | | | | | | | - Minor refactor Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | Merge pull request #29533 from ↵Christoph Wurst2021-11-043-8/+24
|\ \ | |/ |/| | | | | nextcloud/enhancement/get-calendars-for-principal-api Add IManager::getCalendarsForPrincipal API
| * Add IManager::getCalendarsForPrincipal APIChristoph Wurst2021-11-033-8/+24
| | | | | | | | | | | | | | | | The Calendar app needs to access calendars of a given principal in the back-end. The new calendar providers were not accessible for apps before this patch. Now they can access the ICalendar objects on demand. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Merge pull request #29525 from ↵Christoph Wurst2021-11-031-0/+2
|\ \ | |/ |/| | | | | nextcloud/bugfix/noid/allow-to-inject-by-public-interface Allow to inject the trusted domain helper by public interface
| * Allow to inject the trusted domain helper by public interfaceJoas Schilling2021-11-031-0/+2
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | [tx-robot] updated from transifexNextcloud bot2021-11-034-4/+48
|/ | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* Merge pull request #29479 from nextcloud/fix/images-php-8.0Côme Chilliet2021-11-026-23/+26
|\ | | | | Fix image handling for PHP 8.0
| * Fix resource usages in OC_ImageCôme Chilliet2021-10-286-23/+26
| | | | | | | | | | | | This makes sure using resource or GdImage (PHP>=8) behaves the same. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | [tx-robot] updated from transifexNextcloud bot2021-11-022-2/+2
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | Merge pull request #29502 from nextcloud/revert-27378-dav-recalc-checksumJohn Molakvoæ2021-11-011-3/+2
|\ \
| * | Revert "add dav plugin to trigger recalculating of checksums"John Molakvoæ2021-11-011-3/+2
| | |
* | | properly handle cases where cache wrappers block accessRobin Appelman2021-11-012-3/+4
| | | | | | | | | | | | | | | | | | `CacheWrapper::formatCacheEntry` can return false for files that should be filtered out Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #29281 from vijfhoek/masterJohn Molakvoæ2021-11-012-6/+14
|\ \ \ | |/ / |/| |
| * | Move query outside the loop and reduce chunk size to 1000Sijmen Schoon2021-10-172-12/+14
| | | | | | | | | | | | | | | | | | | | | This involved changing CacheQueryBuilder\whereParentIn to take a parameter name, renaming the function accordingly. Signed-off-by: Sijmen Schoon <me@sijmenschoon.nl>
| * | Limit parameter count per query in Cache.removeChildrenSijmen Schoon2021-10-171-8/+14
| | | | | | | | | | | | Signed-off-by: Sijmen Schoon <me@sijmenschoon.nl>
* | | Merge pull request #27378 from nextcloud/dav-recalc-checksumJohn Molakvoæ2021-11-011-2/+3
|\ \ \
| * | | Add dav plugin to trigger recalculating of checksumsRobin Appelman2021-08-231-2/+3
| | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | Register missing DAV app calendar providerChristoph Wurst2021-10-281-1/+2
| |_|/ |/| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Merge pull request #29115 from ↵Carl Schwan2021-10-282-1/+9
|\ \ \ | | | | | | | | | | | | | | | | nextcloud/work/carl/correct-permissions-when-copying Fix permissions when copying from ObjectStorage
| * | | Fix permissions when copying from ObjectStorageCarl Schwan2021-10-282-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure that when a user copy a file from a directory they don't have all permissions to a directory where they have more permissions, the permissions are correctly set to the one from the parent taget folder. This was caused by the ObjectStoreStorage::copyFromStorage using the jailed storage and cache entry instead of the unjailed one like other storages (the local one). Steps to reproduce + Use object storage + Create a groupfolder with one group having full permission and another one who can just read files. + With an user who is in the second group, copy a file from the groupfolder to the home folder of this user. + The file in the home folder of the user will be read only and can't be deleted even though it is in their home folder and they are the owner. In oc_filecache, the permissions stored for this file are 1 (READ) Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | Merge pull request #29444 from ↵Joas Schilling2021-10-284-13/+77
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/bugfix/noid/public-api-for-trusted-domain-helpers Add an OCP for trusted domain helper
| * | | | Add an OCP for trusted domain helperJoas Schilling2021-10-284-13/+77
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | Merge pull request #29453 from nextcloud/bugfix/noid/improve-wordingJohn Molakvoæ2021-10-272-4/+2
|\ \ \ \ \
| * | | | | Improve wordingJoas Schilling2021-10-272-4/+2
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | Prevent unnecesary profile action registrationsChristopher Ng2021-10-261-11/+20
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | | | | Merge pull request #29435 from nextcloud/explicitly-name-profile-link-actionJohn Molakvoæ2021-10-263-11/+11
|\ \ \ \ \
| * | | | | Use more explicit naming for profile link actionChristopher Ng2021-10-253-11/+11
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* / / / / [tx-robot] updated from transifexNextcloud bot2021-10-262-0/+32
|/ / / / | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | | | Merge pull request #29425 from nextcloud/revert-28138-zorn-v-patch-2John Molakvoæ2021-10-251-1/+0
|\ \ \ \
| * | | | Revert "Fix "never catch" catches in OC_App"Joas Schilling2021-10-251-1/+0
| | | | |
* | | | | Merge pull request #26531 from J0WI/refactor-preview-execJohn Molakvoæ2021-10-256-64/+94
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Use findBinaryPath for previewsJ0WI2021-10-236-64/+94
| | | | | | | | | | | | | | | | | | | | Signed-off-by: J0WI <J0WI@users.noreply.github.com>
* | | | | [tx-robot] updated from transifexNextcloud bot2021-10-252-0/+2
|/ / / / | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | | | Add type hint to fread_block $blockSize param & apply cs:fix.alanmeeson2021-10-231-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: alanmeeson <alan@carefullycalculated.co.uk>
* | | | Fix truncation of files upon read when using object store and encryption.Alan Meeson2021-10-231-2/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using and object store as primary storage and using the default encryption module at the same time, any encrypted file would be truncated when read, and a text error message added to the end. This was caused by a combination of the reliance of the read functions on on knowing the unencrypted file size, and a bug in the function which calculated the unencrypted file size for a given file. In order to calculate the unencrypted file size, the function would first skip the header block, then use fseek to skip to the last encrypted block in the file. Because there was a corresponence between the encrypted and unencrypted blocks, this would also be the last encrypted block. It would then read the final block and decrypt it to get the unencrypted length of the last block. With that, the number of blocks, and the unencrypted block size, it could calculate the unencrypted file size. The trouble was that when using an object store, an fread call doesn't always get you the number of bytes you asked for, even if they are available. To resolve this I adapted the stream_read_block function from lib/private/Files/Streams/Encryption.php to work here. This function wraps the fread call in a loop and repeats until it has the entire set of bytes that were requested, or there are no more to get. This fixes the imediate bug, and should (with luck) allow people to get their encrypted files out of Nextcloud now. (The problem was purely on the decryption side). In the future it would be nice to do some refactoring here. I have tested this with image files ranging from 1kb to 10mb using Nextcloud version 22.1.0 (the nextcloud:22.1-apache docker image), with sqlite and a Linode object store as the primary storage. Signed-off-by: Alan Meeson <alan@carefullycalculated.co.uk>
* | | | Merge pull request #27440 from nextcloud/is-file-handleJohn Molakvoæ2021-10-231-4/+4
|\ \ \ \
| * | | | Handle files with is_file instead of file_existsacsfer2021-08-151-4/+4
| | | | | | | | | | | | | | | Should fix things like `fread(): read of 8192 bytes failed with errno=21 Is a directory`
* | | | | Merge pull request #28138 from nextcloud/zorn-v-patch-2John Molakvoæ2021-10-231-0/+1
|\ \ \ \ \
| * | | | | Fix "never catch" catches in OC_Appzorn-v2021-10-221-0/+1
| | | | | | | | | | | | | | | | | | Need to check another legacy. Need to explicitly add to use or do not forget add slash `\`
* | | | | | Merge pull request #29363 from nextcloud/fair-use-pushJohn Molakvoæ2021-10-235-33/+93
|\ \ \ \ \ \
| * | | | | | Fair use of push notificationsJoas Schilling2021-10-235-33/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to keep offering our push notification service for free, but large users overload our infrastructure. For this reason we have to rate-limit the use of push notifications. If you need this feature, consider setting up your own push server or using Nextcloud Enterprise. Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | | | Merge pull request #29397 from nextcloud/fix/noid/profile-missing-def-valuesJohn Molakvoæ2021-10-231-6/+7
|\ \ \ \ \ \ \